Conversation
jacek-lewandowski
approved these changes
Jul 28, 2021
|
Kudos, SonarCloud Quality Gate passed!
|
jacek-lewandowski
pushed a commit
that referenced
this pull request
Mar 8, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Mar 9, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Mar 30, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Mar 30, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Apr 19, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
May 26, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Oct 17, 2022
jacek-lewandowski
pushed a commit
that referenced
this pull request
Oct 18, 2022
mfleming
pushed a commit
that referenced
this pull request
Jul 10, 2023
djatnieks
added a commit
that referenced
this pull request
Jul 24, 2023
djatnieks
added a commit
that referenced
this pull request
Jan 16, 2024
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
djatnieks
added a commit
that referenced
this pull request
Mar 29, 2024
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
djatnieks
added a commit
that referenced
this pull request
Apr 1, 2024
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
djatnieks
added a commit
that referenced
this pull request
Apr 16, 2024
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
djatnieks
added a commit
that referenced
this pull request
Jan 30, 2025
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
djatnieks
added a commit
that referenced
this pull request
May 18, 2025
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 6, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 10, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 11, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 12, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 14, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 16, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Feb 27, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Mar 2, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Mar 4, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Mar 25, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Mar 27, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
michaelsembwever
pushed a commit
that referenced
this pull request
Apr 14, 2026
…227) (cherry picked from commit 0969332) (cherry picked from commit 91a753d) (cherry picked from commit 56a9b56) (cherry picked from commit a2c6bb4) (cherry picked from commit 1bd3f63) (cherry picked from commit 0449b6c) (cherry picked from commit 217089a) # Conflicts: # test/unit/org/apache/cassandra/cql3/CQLTester.java # test/unit/org/apache/cassandra/guardrails/GuardrailTester.java (Rebase of commit eb00759)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
STAR-847 guardrail unit tests sometimes fail with
User guardrail_user has no SELECT permission on <table cql_test_keyspace.table_01> or any of its parentsThe
GuardrailTester. afterGuardrailTestmethod drops the test user after each test, however the java driver cluster/session remains open. If the timing is right, the driver control connection will issue queries as the test user after the user has been dropped, and this can result in an empty entry in the Roles cache. If that entry does not expire before the next test starts and the test user is created again, the empty entry will result in no permissions being recognized for the user.A previous attempt to fix the issue - issuing a LIST ROLES query and waiting for results - didn't actually go through the Roles cache, so wasn't effective and is removed here.
This fix closes the driver cluster/session belonging to the test user when the test user is dropped, preventing bad entries from getting into the Roles cache.