Skip to content

CNDB-16886: Reject BM25 queries in validation when the coordinator is before EC (#2459)#2483

Open
djatnieks wants to merge 1 commit into
main-5.0from
CNDB-18271
Open

CNDB-16886: Reject BM25 queries in validation when the coordinator is before EC (#2459)#2483
djatnieks wants to merge 1 commit into
main-5.0from
CNDB-18271

Conversation

@djatnieks

Copy link
Copy Markdown
Member

Reject BM25 queries on Index.validate(ReadCommand) if the current
index version doesn't support them. That way they would be rejected on
the coordinator before execution, which is faster and produces a
descriptive error message, rather than the generic per-replica error
codes. The replicas still will do their own validation if the query gets
past the coordinator.

@djatnieks

djatnieks commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

resolved trivial import conflict

@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)

… before EC (#2459)

Reject BM25 queries on `Index.validate(ReadCommand)` if the current
index version doesn't support them. That way they would be rejected on
the coordinator before execution, which is faster and produces a
descriptive error message, rather than the generic per-replica error
codes. The replicas still will do their own validation if the query gets
past the coordinator.
@plpesvc-ds

Copy link
Copy Markdown

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