Skip to content

CNDB-15608 add and improve some SAI related tests#2287

Open
k-rus wants to merge 2 commits intomainfrom
rf-complex-schema-test-from-15608
Open

CNDB-15608 add and improve some SAI related tests#2287
k-rus wants to merge 2 commits intomainfrom
rf-complex-schema-test-from-15608

Conversation

@k-rus
Copy link
Copy Markdown
Member

@k-rus k-rus commented Mar 27, 2026

What is the issue

Part of https://github.com/riptano/cndb/issues/15608

The patch in https://github.com/riptano/cndb/issues/15608 adds new tests and improves existing tests and such changes can be already applied in main independently from the patch.

What does this PR fix and why was it fixed

Adds a test coming from latte benchmark, where it was used for investigating performance on a complex schema with SAIs based on a real world scenario. The test run into a bug in the patch in CNDB-15608, which was fixed, and is added to prevent regressions.

Improve test of numeric values store by changing to use larger values for tokens.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

Checklist before you submit for review

  • This PR adheres to the Definition of Done
  • Make sure there is a PR 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 DataStax copyright header instead of the Apache License one

@k-rus k-rus changed the title CNDB-15608 add a complex SAI test reproducing a bug CNDB-15608 add and improve some SAI related tests Mar 27, 2026
@k-rus
Copy link
Copy Markdown
Member Author

k-rus commented Mar 27, 2026

  • Make sure there is a PR in the CNDB project updating the Converged Cassandra version

This PR only affects tests, this no CNBD PR will be created.

k-rus added 2 commits April 13, 2026 15:30
This test is coming from latte benchmark, where it was used for
investigating performance on a complex schema with SAIs based on a
real world scenario. The test run into a bug in the patch in
CNDB-15608, which was fixed, and is added to prevent regressions.
Changes to use larger values for tokens.
@k-rus k-rus force-pushed the rf-complex-schema-test-from-15608 branch from 006d636 to 289e487 Compare April 13, 2026 13:30
@sonarqubecloud
Copy link
Copy Markdown

@cassci-bot
Copy link
Copy Markdown

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


3 regressions found
See build details here


Found 3 new test failures

Test Explanation Runs Upstream
o.a.c.index.sai.cql.VectorCompaction100dTest.testZeroOrOneToManyCompaction[version=ec enableNVQ=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorKeyRestrictedOnPartitionTest.partitionRestrictedWidePartitionBqCompressedTest[version=ca enableNVQ=false enableFused=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testRerankKZeroOrderMatchesFullPrecisionSimilarity[version=ed enableNVQ=true enableFused=false] REGRESSION 🔴 0 / 30

Found 9 known test failures

@k-rus k-rus requested a review from a team April 13, 2026 18:07
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.

2 participants