Skip to content

DO NOT MERGE. Duplicate of https://github.com/datastax/cassandra/pull/2292 for test purposes#2293

Closed
ekaterinadimitrova2 wants to merge 2 commits intomainfrom
cndb-17161-20260-test
Closed

DO NOT MERGE. Duplicate of https://github.com/datastax/cassandra/pull/2292 for test purposes#2293
ekaterinadimitrova2 wants to merge 2 commits intomainfrom
cndb-17161-20260-test

Conversation

@ekaterinadimitrova2
Copy link
Copy Markdown

What is the issue

...
CI doesn't start for #2292 so I am testing a brand new PR.

Tried on the other PR to:

  • commit empty commit
  • close the PR and reopen it
  • marked as draft and back to ready
    Nothing helped, so testing a brand new PR

What does this PR fix and why was it fixed

...

ekaterinadimitrova2 and others added 2 commits March 16, 2026 16:34
…leIndexWriteLatency, termsTraversalTotalTime, intersectionLatency, queryLatency (#2267)

### What is the issue
...
We want to be able to disable SAI histogram metrics
### What does this PR fix and why was it fixed
...
Add flag to enable/disable SAI histogram metrics -
`memtableIndexWriteLatency, termsTraversalTotalTime,
intersectionLatency`, `queryLatency`

`-Dcassandra.sai.metrics.histograms.enabled`
The metrics are kept enabled by default as they were before.

---------

Co-authored-by: Andrés de la Peña <adelapena@users.noreply.github.com>
@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 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

@ekaterinadimitrova2
Copy link
Copy Markdown
Author

Main PR was unblocked. Closing this one as we won't need it.

@ekaterinadimitrova2 ekaterinadimitrova2 deleted the cndb-17161-20260-test branch March 30, 2026 20:14
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