Skip to content

CNDB-17161-main: Added method to wait index builds at CFS initialization and fix SSTableContextManager#sstableDescriptors cache on SAI initialization#2302

Merged
jasonstack merged 2 commits intomainfrom
cndb-17161-main
Apr 6, 2026
Merged

CNDB-17161-main: Added method to wait index builds at CFS initialization and fix SSTableContextManager#sstableDescriptors cache on SAI initialization#2302
jasonstack merged 2 commits intomainfrom
cndb-17161-main

Conversation

@jasonstack
Copy link
Copy Markdown

@jasonstack jasonstack commented Apr 2, 2026

What is the issue

CNDB-17161

What does this PR fix and why was it fixed

Added method to wait index builds at CFS initialization and fix SSTableContextManager#sstableDescriptors cache on SAI initialization

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 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

@jasonstack jasonstack force-pushed the cndb-17161-main branch 3 times, most recently from 27ca136 to c11efc3 Compare April 3, 2026 07:21
@jasonstack
Copy link
Copy Markdown
Author

Not sure why sonarqubecloud doesn't find the test coverage.. @pkolaczk is there a walkaround?

@pkolaczk
Copy link
Copy Markdown

pkolaczk commented Apr 3, 2026

@jasonstack Yesterday I had the same issue with another PR. I don't know a workaround other than:
checking coverage manually (in IDE) and then bypassing the merge rules.

@jasonstack
Copy link
Copy Markdown
Author

@pkolaczk the release PR has 100% coverage in #2292 (comment). Do you have permission to bypass the gate?

sbtourist and others added 2 commits April 6, 2026 09:49
…ndle different index sets to avoid returning IndexDescriptor for indexes that have not be loaded

- Use CHM for IndexDescriptor#perIndexes

- only load missing indexes
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 6, 2026

@cassci-bot
Copy link
Copy Markdown

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


5 regressions found
See build details here


Found 5 new test failures

Test Explanation Runs Upstream
o.a.c.distributed.test.sai.SlowSAIQueryLoggerTest.testSlowSAIQueryLogger REGRESSION 🔴🔴 2 / 30
o.a.c.index.sai.cql.VectorCompaction100dTest.testZeroOrOneToManyCompaction[version=ec enableNVQ=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testRerankKZeroOrderMatchesFullPrecisionSimilarity[version=fa enableNVQ=false enableFused=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorTypeTest.nullVectorTest[version=ca enableNVQ=false enableFused=false] REGRESSION 🔴 0 / 30
o.a.c.utils.binlog.BinLogTest.testTruncationReleasesLogSpace (compression) REGRESSION 🔵🔴 0 / 30

Found 9 known test failures

@jasonstack jasonstack merged commit f50bc93 into main Apr 6, 2026
2 of 4 checks passed
@jasonstack jasonstack deleted the cndb-17161-main branch April 6, 2026 03:25
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