Skip to content

Conversation

@5
Copy link

@5 5 commented Mar 28, 2023

Node draining aborts all current sstables imports

SS tables imports that will end up being ignored due to the node draining fail instead of succeeding.
Each active SS tables import periodically checks for the node status and sees whether or not it is DRAINING.
In case the node starts draining the import aborts immediately by throwing an InterruptedException.

patch by Yuriy Semchyshyn; reviewed by Yifan Cai for CASSANDRA-18373

@5 5 requested a review from yifan-c April 4, 2023 00:08
Copy link
Member

@dineshjoshi dineshjoshi left a comment

Choose a reason for hiding this comment

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

LGTM. Minor nits.

@5 5 requested a review from dineshjoshi April 5, 2023 13:39
Yuriy Semchyshyn added 3 commits April 5, 2023 12:16
SSTables imports that have no chance of succeeding due to the node being drained fail fast.
In case there are any sstables imports currently active, a draining node notifies all of them upon flushing its column family stores.
Each notified sstables import aborts as soon as possible by throwing an `InterruptedException`.

patch by Yuriy Semchyshyn; reviewed by Yifan Cai for CASSANDRA-18373
@smiklosovic smiklosovic closed this Apr 9, 2023
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