Skip to content

CNDB-15608 port KeyRangeConcatIteratorTest refactoring#2268

Merged
k-rus merged 2 commits intomainfrom
rf-key-range-concat-test-refactor
Mar 16, 2026
Merged

CNDB-15608 port KeyRangeConcatIteratorTest refactoring#2268
k-rus merged 2 commits intomainfrom
rf-key-range-concat-test-refactor

Conversation

@k-rus
Copy link
Copy Markdown
Member

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

What is the issue

KeyRangeConcatIteratorTest can be improved and is part of Apache patch port in https://github.com/riptano/cndb/issues/15608.

What does this PR fix and why was it fixed

This refactoring improves the test and is isolated from the other patch port, so it make sense to do it separately.

This refactoring improves the test and is isolated from the other patch
port, so it make sense to do it separately.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 12, 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
Copy link
Copy Markdown
Member Author

k-rus commented Mar 12, 2026

@k-rus k-rus requested a review from a team March 12, 2026 13:10
@sonarqubecloud
Copy link
Copy Markdown

@cassci-bot
Copy link
Copy Markdown

❌ Build ds-cassandra-pr-gate/PR-2268 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[ec false] REGRESSION 🔴 0 / 25
o.a.c.index.sai.cql.VectorSiftSmallTest.testMultiSegmentBuild[ec true false] REGRESSION 🔴🔴 0 / 25
o.a.c.index.sai.cql.types.collections.maps.MapTimeTest.test[version=eb,dataset=map<time,time>,wide=false,scenario=POST_BUILD_QUERY] (compression) REGRESSION 🔴🔵 0 / 25

Found 5 known test failures

@k-rus k-rus merged commit 46052f1 into main Mar 16, 2026
2 of 4 checks passed
@k-rus k-rus deleted the rf-key-range-concat-test-refactor branch March 16, 2026 09:41
scottfines pushed a commit that referenced this pull request Apr 9, 2026
This refactoring improves the test and is isolated from the other patch
port, so it make sense to do it separately.
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