Skip to content

Optimize ACL Parser Performance by Conditional Application of Recursive Analysis #1226

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 27, 2025
Merged

Optimize ACL Parser Performance by Conditional Application of Recursive Analysis #1226

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
563c356
ACL: Optimize ACL Parser to Avoid Long Running Invocations
May 26, 2025
d4c37f2
Add optimization to pass tests (TDD).
May 26, 2025
e1d3178
Fix newline formatting
May 26, 2025
a96ce2f
Address copilot PR feedback + additional test case
May 27, 2025
32684c2
Address PR feedback
May 27, 2025
96e74bf
Undo accessibility change
May 27, 2025
072c873
Accessibility change
May 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Undo accessibility change
  • Loading branch information
kbowersox committed May 27, 2025
commit 96e74bf8d846b33b7ffc9a540bba59ab502cee04
2 changes: 1 addition & 1 deletion test/Garnet.test/Resp/ACL/AclParserTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ namespace Garnet.test.Resp.ACL
/// Tests for the <see cref="ACLParser"/>.
/// </summary>
[TestFixture]
public class AclParserTests : AclTest
class AclParserTests : AclTest
{
/// <summary>
/// Test cases for scenarios where ACLs reduce.
Expand Down
Loading