Skip to content

CNDB-18062 test row aware primary key map with static clustering (#2444)#2480

Merged
djatnieks merged 1 commit into
main-5.0from
CNDB-18267
Jul 1, 2026
Merged

CNDB-18062 test row aware primary key map with static clustering (#2444)#2480
djatnieks merged 1 commit into
main-5.0from
CNDB-18267

Conversation

@djatnieks

@djatnieks djatnieks commented Jun 22, 2026

Copy link
Copy Markdown
Member

Static clustering is a special case, and, thus, important to exercise in tests. Row aware primary key map API is not exercises in the case of static clustering as it was discovered during work on CNDB-15608.

Adds unit tests on row aware primary key map API with static clustering.

Moves parameterizing on row aware versions into CQL tester.

Static clustering is a special case, and, thus, important to exercise in
tests. Row aware primary key map API is not exercises in the case of
static clustering as it was discovered during work on CNDB-15608.

Adds unit tests on row aware primary key map API with static clustering.

Moves parameterizing on row aware versions into CQL tester.
@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)

@djatnieks

Copy link
Copy Markdown
Member Author

Clean merge

@plpesvc-ds

Copy link
Copy Markdown

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


4 regressions found
See build details here


Found 4 new test failures

Test Explanation Runs Upstream
junit.framework.TestSuite.org.apache.cassandra.distributed.test.sai.features.FeaturesVersionSupportAATest-_jdk11 REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.GeoDistanceAccuracyTest.haversineBenchmark (compression) REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorCompaction100dTest.testPQRefine[version=ec enableNVQ=false] () NEW 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testMultiSegmentBuild[ed false] () NEW 🔴 0 / 30

Found 1 known test failures

@djatnieks djatnieks merged commit 44ba75a into main-5.0 Jul 1, 2026
4 of 6 checks passed
@djatnieks djatnieks deleted the CNDB-18267 branch July 1, 2026 18:30
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.

4 participants