Skip to content

RATIS-2492. Bump version to 1.1.0-SNAPSHOT#104

Merged
adoroszlai merged 1 commit intoapache:masterfrom
adoroszlai:RATIS-2492
Apr 3, 2026
Merged

RATIS-2492. Bump version to 1.1.0-SNAPSHOT#104
adoroszlai merged 1 commit intoapache:masterfrom
adoroszlai:RATIS-2492

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

RATIS-2491 proposes upgrade of Netty from 4.1 to 4.2. There may be compatibility issues, so I think we should incorporate it in ratis-thirdparty-1.1.0, and maintain ratis-thirdparty-1.0.x for security fixes on Netty 4.1 (and other dependencies).

This PR bumps project version number to 1.1.0-SNAPSHOT. Afterwards we can create the maintenance branch from the previous commit. Would release-1.0.x be a good name for the branch?

https://issues.apache.org/jira/browse/RATIS-2492

How was this patch tested?

https://github.com/adoroszlai/ratis-thirdparty/actions/runs/23905713849

@szetszwo
Copy link
Copy Markdown
Contributor

szetszwo commented Apr 3, 2026

For compatibility issues, we probably should update major release, i.e. 2.0.0.

Then, we can create a new branch "branch-1" for versions 1.x.x.

Since there are many version upgrade (thanks dependabot)), let's also bump the branch-1 version to 1.1.0.

@adoroszlai
Copy link
Copy Markdown
Contributor Author

Since there are many version upgrade (thanks dependabot)), let's also bump the branch-1 version to 1.1.0.

Thanks @szetszwo for the comments. Then I think this PR is OK, and we can post a follow-up PR to bump to 2.0.0-SNAPSHOT.

Copy link
Copy Markdown
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

@adoroszlai , you are right that this PR is good to go.

+1 the change looks good.

@adoroszlai adoroszlai merged commit 680b495 into apache:master Apr 3, 2026
3 checks passed
@adoroszlai adoroszlai deleted the RATIS-2492 branch April 3, 2026 17:42
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @szetszwo for the review.

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