Skip to content

Align BatchWriteItem affinity query plan#100

Open
dkropachev wants to merge 2 commits into
mainfrom
dkropachev/driver-659-batchwrite-affinity-regression
Open

Align BatchWriteItem affinity query plan#100
dkropachev wants to merge 2 commits into
mainfrom
dkropachev/driver-659-batchwrite-affinity-regression

Conversation

@dkropachev

@dkropachev dkropachev commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Fixes #67

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

Summary:

  • Return every BatchWriteItem voted live coordinator ordered by vote count, then node address.
  • Try voted nodes first and keep canonical retry order for remaining nodes.
  • Preserve invalid PutRequest/DeleteRequest union filtering already added on this branch.

Tests:

  • uv run pytest tests/unit/test_key_affinity.py -q
  • make lint

@github-actions github-actions Bot added the test label Jun 30, 2026
@dkropachev
dkropachev force-pushed the dkropachev/driver-659-batchwrite-affinity-regression branch from e0a468b to 867780b Compare June 30, 2026 17:15
@dkropachev dkropachev changed the title Ignore invalid BatchWriteItem affinity candidates Align BatchWriteItem affinity query plan Jun 30, 2026
@dkropachev
dkropachev requested a review from Copilot June 30, 2026 18:08

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cross-language deterministic BatchWriteItem key-affinity candidate selection

2 participants