Skip to content

feat(rbac): allocation-free storage#23

Open
m62624 wants to merge 42 commits intomasterfrom
refactor/access-control
Open

feat(rbac): allocation-free storage#23
m62624 wants to merge 42 commits intomasterfrom
refactor/access-control

Conversation

@m62624
Copy link
Member

@m62624 m62624 commented Feb 10, 2026

Resolves #21

@m62624 m62624 changed the base branch from feat/rbac-benchmarks to master February 23, 2026 11:35
@m62624 m62624 marked this pull request as ready for review February 27, 2026 08:22
@m62624 m62624 requested a review from vobradovich February 27, 2026 08:22
@m62624 m62624 marked this pull request as draft February 27, 2026 08:29
@m62624 m62624 removed the request for review from vobradovich February 27, 2026 08:32
@gear-tech gear-tech deleted a comment from github-actions bot Feb 27, 2026
@gear-tech gear-tech deleted a comment from github-actions bot Feb 27, 2026
@gear-tech gear-tech deleted a comment from github-actions bot Feb 27, 2026
@github-actions
Copy link
Contributor

🔬 Click to see benchmark results vs master

🔬 Benchmark Comparison

Metric Current Baseline Change % Status
access_control.grant_role.0 940_239_148 990_713_476 -50_474_328 -5.09% 🚀
access_control.grant_role.100 827_111_188 852_534_024 -25_422_836 -2.98% 👍
access_control.grant_role.1000 1_451_020_927 1_076_492_777 +374_528_150 34.79%
access_control.grant_role.500 1_124_708_432 - - -
access_control.has_role.0 956_977_723 837_548_725 +119_428_998 14.26%
access_control.has_role.100 812_240_826 850_763_647 -38_522_821 -4.53% 👍
access_control.has_role.1000 855_644_080 956_654_964 -101_010_884 -10.56% 🚀
access_control.has_role.500 854_463_460 - - -
access_control.has_role_multi.1 956_977_723 837_548_725 +119_428_998 14.26%
access_control.has_role_multi.10 842_943_738 839_663_860 +3_279_878 0.39%
access_control.has_role_multi.100 868_681_967 881_167_357 -12_485_390 -1.42% 👍
access_control.has_role_multi.50 883_202_733 873_880_080 +9_322_653 1.07% ⚠️
access_control.revoke_role.0 966_419_663 844_228_279 +122_191_384 14.47%
access_control.revoke_role.100 831_802_725 857_192_808 -25_390_083 -2.96% 👍
access_control.revoke_role.1000 1_455_998_094 1_079_844_475 +376_153_619 34.83%
access_control.revoke_role.500 1_129_521_472 - - -

Legend

  • 🚀 Significant improvement (>5% reduction)
  • 👍 Minor improvement (<1% reduction)
  • ✅ No significant change
  • ⚠️ Minor regression (>1% increase)
  • ❌ Significant regression (>5% increase)
  • ✨ New metric (not present in baseline)
  • 🗑️ Removed metric (not present in current)

🤖 This comment was automatically generated. Baseline: branch master.

@gear-tech gear-tech deleted a comment from github-actions bot Feb 27, 2026
@m62624 m62624 force-pushed the refactor/access-control branch from a77d287 to 515a858 Compare February 27, 2026 19:21
@github-actions
Copy link
Contributor

🔬 Click to see benchmark results vs master

🔬 Benchmark Comparison

Metric Current Baseline Change % Status
access_control.grant_role.0 960_438_002 990_713_476 -30_275_474 -3.06% 👍
access_control.grant_role.100 966_864_469 852_534_024 +114_330_445 13.41%
access_control.grant_role.1000 968_890_515 1_076_492_777 -107_602_262 -10.00% 🚀
access_control.grant_role.10000 984_539_185 1_178_622_849 -194_083_664 -16.47% 🚀
access_control.grant_role.5000 976_270_007 1_172_191_147 -195_921_140 -16.71% 🚀
access_control.has_role.0 781_166_252 837_548_725 -56_382_473 -6.73% 🚀
access_control.has_role.100 846_332_241 850_763_647 -4_431_406 -0.52%
access_control.has_role.1000 846_691_491 956_654_964 -109_963_473 -11.49% 🚀
access_control.has_role.10000 847_769_241 1_062_272_047 -214_502_806 -20.19% 🚀
access_control.has_role.5000 847_409_991 1_053_185_958 -205_775_967 -19.54% 🚀
access_control.has_role_multi.1 781_166_252 837_548_725 -56_382_473 -6.73% 🚀
access_control.has_role_multi.10 861_440_642 839_663_860 +21_776_782 2.59% ⚠️
access_control.has_role_multi.100 926_961_672 881_167_357 +45_794_315 5.20%
access_control.has_role_multi.50 926_723_575 873_880_080 +52_843_495 6.05%
access_control.revoke_role.0 789_929_837 844_228_279 -54_298_442 -6.43% 🚀
access_control.revoke_role.100 970_519_711 857_192_808 +113_326_903 13.22%
access_control.revoke_role.1000 972_129_673 1_079_844_475 -107_714_802 -9.98% 🚀
access_control.revoke_role.10000 987_799_063 1_183_281_633 -195_482_570 -16.52% 🚀
access_control.revoke_role.5000 979_518_637 1_175_541_661 -196_023_024 -16.68% 🚀

Legend

  • 🚀 Significant improvement (>5% reduction)
  • 👍 Minor improvement (<1% reduction)
  • ✅ No significant change
  • ⚠️ Minor regression (>1% increase)
  • ❌ Significant regression (>5% increase)
  • ✨ New metric (not present in baseline)
  • 🗑️ Removed metric (not present in current)

🤖 This comment was automatically generated. Baseline: branch master.

@m62624 m62624 force-pushed the refactor/access-control branch from 515a858 to 3666c35 Compare March 3, 2026 05:58
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

🔬 Click to see benchmark results vs master

🔬 Benchmark Comparison

Metric Current Baseline Change % Status
access_control.grant_role.0 940_442_455 990_713_476 -50_271_021 -5.07% 🚀
access_control.grant_role.100 840_398_213 852_534_024 -12_135_811 -1.42% 👍
access_control.grant_role.25 808_699_805 - - -
access_control.grant_role.50 819_345_109 - - -
access_control.has_role.0 945_877_394 837_548_725 +108_328_669 12.93%
access_control.has_role.100 793_087_438 850_763_647 -57_676_209 -6.78% 🚀
access_control.has_role.25 792_612_430 - - -
access_control.has_role.50 792_849_934 - - -
access_control.has_role_multi.1 945_877_394 837_548_725 +108_328_669 12.93%
access_control.has_role_multi.16 832_081_506 - - -
access_control.has_role_multi.32 832_319_603 - - -
access_control.has_role_multi.8 832_214_209 - - -
access_control.revoke_role.0 955_319_800 844_228_279 +111_091_521 13.16%
access_control.revoke_role.100 844_334_507 857_192_808 -12_858_301 -1.50% 👍
access_control.revoke_role.25 812_591_699 - - -
access_control.revoke_role.50 823_251_803 - - -

Legend

  • 🚀 Significant improvement (>5% reduction)
  • 👍 Minor improvement (<1% reduction)
  • ✅ No significant change
  • ⚠️ Minor regression (>1% increase)
  • ❌ Significant regression (>5% increase)
  • ✨ New metric (not present in baseline)
  • 🗑️ Removed metric (not present in current)

🤖 This comment was automatically generated. Baseline: branch master.

@m62624 m62624 marked this pull request as ready for review March 3, 2026 06:16
@m62624 m62624 requested a review from vobradovich March 3, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RBAC allocation-free storage

1 participant