Skip to content

CNDB-17791: measure only active memtable cleanup windows in FlushFailingOnNotificationSubscriberTest#2418

Merged
driftx merged 1 commit into
main-5.0from
CNDB-17791
May 14, 2026
Merged

CNDB-17791: measure only active memtable cleanup windows in FlushFailingOnNotificationSubscriberTest#2418
driftx merged 1 commit into
main-5.0from
CNDB-17791

Conversation

@driftx
Copy link
Copy Markdown

@driftx driftx commented May 13, 2026

What is the issue

The test is flaky

What does this PR fix and why was it fixed

Track maxTimeSinceCleanup only while the memory pool actually needs cleaning, and reset the window once cleanup completes or after a successful flush.

…ingOnNotificationSubscriberTest

Track maxTimeSinceCleanup only while the memory pool actually needs
cleaning, and reset the window once cleanup completes or after a
successful flush.
@github-actions
Copy link
Copy Markdown

Checklist before you submit for review

  • This PR adheres to the Definition of Done
  • Make sure there is a PR and ticket in the CNDB project updating the Converged Cassandra version
  • Use NoSpamLogger for log lines that may appear frequently in the logs
  • Verify test results on Butler
  • Test coverage for new/modified code is > 80%
  • Proper code formatting
  • Proper title for each commit staring with the project-issue number, like CNDB-1234
  • Each commit has a meaningful description
  • Each commit is not very long and contains related changes
  • Renames, moves and reformatting are in distinct commits
  • All new files should contain the IBM copyright header instead of the Apache License one (no DataStax copyright any longer)

@sonarqubecloud
Copy link
Copy Markdown

@plpesvc-ds
Copy link
Copy Markdown

❌ Build ds-cassandra-pr-gate/PR-2418 rejected by Butler


2 regressions found
See build details here


Found 2 new test failures

Test Explanation Runs Upstream
o.a.c.index.sai.cql.VectorCompaction100dTest.testOneToManyCompaction[version=ec enableNVQ=true] () NEW 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testRerankKZeroOrderMatchesFullPrecisionSimilarity[version=ed enableNVQ=true enableFused=false] () NEW 🔴 0 / 30

Found 3 known test failures

@driftx driftx merged commit b72e056 into main-5.0 May 14, 2026
5 of 7 checks passed
@driftx driftx deleted the CNDB-17791 branch May 14, 2026 15:41
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.

3 participants