Skip to content

Conversation

@zhjwpku
Copy link
Contributor

@zhjwpku zhjwpku commented Mar 24, 2022

Signed-off-by: 元组 [email protected]

元组 added 2 commits March 29, 2022 16:53
serialize `isSuspended` field, and modify the test case to
enable Snapshot failover.

Signed-off-by: 元组 <[email protected]>
@leonardBang leonardBang self-requested a review March 31, 2022 03:00
this.sourceConfig = sourceConfig;
this.splitAssigner = splitAssigner;
this.readersAwaitingSplit = new TreeSet<>();
if (splitAssigner.getAssignerStatus() == NEWLY_ADDED_ASSIGNING
Copy link
Contributor

Choose a reason for hiding this comment

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

We can use AssignerStatus.isAssigning(splitAssigner) || AssignerStatus.isAssigningFinished(splitAssigner) to cover newly assigning and initial assigning.

Signed-off-by: 元组 <[email protected]>
Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @zhjwpku for the update, LGTM

@leonardBang leonardBang merged commit eb230bf into apache:master Apr 7, 2022
@leonardBang
Copy link
Contributor

@zhjwpku Could you also open a PR for release-2.2 branch?

@zhjwpku
Copy link
Contributor Author

zhjwpku commented Apr 7, 2022

yeah, see #1055

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