Skip to content
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.

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

C16971 4.0 #180

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

C16971 4.0 #180

wants to merge 1 commit into from

Conversation

ekaterinadimitrova2
Copy link
Owner

No description provided.

ekaterinadimitrova2 pushed a commit that referenced this pull request May 30, 2024
Co-authored-by: Zhao Yang <[email protected]>
Co-authored-by: Paulo Ricardo Motta Gomes <[email protected]>
Co-authored-by: Eduard Tudenhöfner <[email protected]>
Co-authored-by: Andrés de la Peña <[email protected]>
Co-authored-by: Benjamin Lerer <[email protected]>
(cherry picked from commit a83e0e8)
(cherry picked from commit b90b433)
(cherry picked from commit 36600df)
(cherry picked from commit abc39d6)
(cherry picked from commit 7a263dc)
ekaterinadimitrova2 pushed a commit that referenced this pull request May 30, 2024
Co-authored-by: Zhao Yang <[email protected]>
Co-authored-by: Paulo Ricardo Motta Gomes <[email protected]>
Co-authored-by: Eduard Tudenhöfner <[email protected]>
Co-authored-by: Andrés de la Peña <[email protected]>
Co-authored-by: Benjamin Lerer <[email protected]>
(cherry picked from commit a83e0e8)
(cherry picked from commit b90b433)
(cherry picked from commit 36600df)
(cherry picked from commit abc39d6)
(cherry picked from commit 7a263dc)
(cherry picked from commit 159a1fa)
(cherry picked from commit 5037674)
ekaterinadimitrova2 pushed a commit that referenced this pull request Jun 3, 2024
Co-authored-by: Zhao Yang <[email protected]>
Co-authored-by: Paulo Ricardo Motta Gomes <[email protected]>
Co-authored-by: Eduard Tudenhöfner <[email protected]>
Co-authored-by: Andrés de la Peña <[email protected]>
Co-authored-by: Benjamin Lerer <[email protected]>
(cherry picked from commit a83e0e8)
(cherry picked from commit b90b433)
(cherry picked from commit 36600df)
(cherry picked from commit abc39d6)
(cherry picked from commit 7a263dc)
(cherry picked from commit 159a1fa)
(cherry picked from commit 5037674)
(cherry picked from commit ab880d1)

STAR-543 Fix rebase compile errors and adapt new guardrails for OS 5.0 implementation;
Ported CC implementation of guardrails to OS - loggedBatchEnabled, scannedTombstones, batchSize, and unloggedBatchAcrossPartitions;
GuardedCounter has been ported from CC and needed a temporary fix to support the situation in ReadCommand where a ClientState is not readily available;
From CC, the GuardrailLoggedBatchTest has been added to o.a.c.db.guardrails package

STAR-543 Fix ClientWarningsTest.testTombstoneWarning

STAR-543 Slightly better fix to allow GuardedCounter.add to trigger a failure and raise a GuardrailViolatedException.

STAR-543 Ignore TombstoneCountWarningTest because scannedTombstones guardrail replaces tombstone warnings from CASSANDRA-16896

STAR-543 Add cassandra.enable_guardrails_for_anonymous_user property to allow CC to enable guardrails when authentication is not used.
In GuardrailCollectionSizeOnSSTableWriterTest.testMapSize ensure inserted values are unique to bypass NoSpamLogger shouldLog checks.

STAR-543 Fix GuardrailWriteConsistencyLevelsTest failures
Remove guardrail checks from ConsistencyLevel that duplicate guardrail checks already in Statement classes;
Change expectations in GuardrailWriteConsistencyLevelsTest.testLWTQuery when null serial CL is used to reflect the ConsistencyLevel.defaultSerialConsistency method results.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant