Skip to content

Align BatchWriteItem affinity query plan#67

Open
dkropachev wants to merge 1 commit into
mainfrom
dkropachev/driver-658-batchwrite-affinity-query-plan
Open

Align BatchWriteItem affinity query plan#67
dkropachev wants to merge 1 commit into
mainfrom
dkropachev/driver-658-batchwrite-affinity-query-plan

Conversation

@dkropachev

Copy link
Copy Markdown
Collaborator

Fixes #66

Jira: https://scylladb.atlassian.net/browse/DRIVER-658
Epic: https://scylladb.atlassian.net/browse/DRIVER-653

Summary:

  • Add support for multiple preferred nodes in AlternatorQueryPlan while keeping single-node constructor usage working.
  • Order BatchWriteItem voted nodes by vote count, then coordinator URL.
  • Ignore malformed BatchWriteItem entries that contain both PutRequest and DeleteRequest.

Tests:

  • npm run typecheck
  • npm run lint
  • npm run test

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

Cross-language deterministic BatchWriteItem key-affinity candidate selection

2 participants