Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CELEBORN-1499] Bump Ratis version from 3.0.1 to 3.1.0 #2610

Closed
wants to merge 1 commit into from

Conversation

SteNicholas
Copy link
Member

@SteNicholas SteNicholas commented Jul 10, 2024

What changes were proposed in this pull request?

Bump Ratis version from 3.0.1 to 3.1.0. Meanwhile, remove CelebornStateMachineStorage with the release of apache/ratis#1111.

Why are the changes needed?

Bump Ratis version from 3.0.1 to 3.1.0. Ratis has released v3.1.0, of which release note refers to 3.1.0. The 3.1.0 version is a minor release with multiple improvements and bugfixes including [RATIS-2111] Reinitialize should load the latest snapshot. See the changes between 3.0.1 and 3.1.0 releases.

Follow up #2547.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

MasterStateMachineSuiteJ#testInstallSnapshot

@SteNicholas
Copy link
Member Author

SteNicholas commented Jul 10, 2024

Ping @AngersZhuuuu, @leixm, @RexXiong, @FMX.

Copy link
Contributor

@RexXiong RexXiong left a comment

Choose a reason for hiding this comment

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

LGTM

@FMX
Copy link
Contributor

FMX commented Jul 11, 2024

I think this PR should test rolling upgrade scenerio.

@leixm
Copy link
Contributor

leixm commented Jul 11, 2024

LGTM

Copy link
Contributor

@AngersZhuuuu AngersZhuuuu left a comment

Choose a reason for hiding this comment

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

LGTM

@SteNicholas
Copy link
Member Author

SteNicholas commented Jul 11, 2024

@FMX, I have tested the rolling upgrade scenerio in test environment as follows:
image
image
The result of rolling upgrade is that there is no compatibility problem between the communication of 3.0.1 ratis server and 3.1.0 ratis server.
image
Meanwhile, I have run a test application successfully based on the above situation of master:
image
image
PTAL.

@RexXiong RexXiong closed this in adbef7b Jul 11, 2024
@RexXiong
Copy link
Contributor

Merge to main(v0.6.0)

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.

None yet

5 participants