Skip to content

Conversation

pjfanning
Copy link
Member

@pjfanning pjfanning commented Nov 5, 2023

idea is to possibly support netty 4 for pekko 1.0 users

@pjfanning pjfanning marked this pull request as draft November 5, 2023 20:32
@pjfanning pjfanning changed the title netty 4 support [DRAFT] netty 4 support Nov 5, 2023
@mdedetrich
Copy link
Contributor

Am I missing something, isn't the whole ideal that if you want netty4 you should use 1.1.x?

@pjfanning
Copy link
Member Author

Am I missing something, isn't the whole ideal that if you want netty4 you should use 1.1.x?

Not if we intend to delay releasing pekko 1.1 for months and months.

@mdedetrich
Copy link
Contributor

mdedetrich commented Nov 5, 2023

Not if we intend to delay releasing pekko 1.1 for months and months.

Then we should release a M1 of the 1.1.x branch. The whole point of 1.0.x is its behaviour is meant to be as similar as possible to Akka 2.6.20 to help users transition and this is an actual commitment that we collectively made, the change in netty 3 to netty 4 completely changes that proposition.

For these reasons I am against this, unless there is a stronger argument that isn't "users cannot wait for a few more months" (I also haven't heard of anyone complaining about netty4 not being available right now).

At the very least this should be discussed on the mailing list.

@pjfanning
Copy link
Member Author

Not if we intend to delay releasing pekko 1.1 for months and months.

Then we should release a M1 of the 1.1.x branch. The whole point of 1.0.x is its behaviour is meant to be as similar as possible to Akka 2.6.20 to help users transition and this is an actual commitment that we collectively made, the change in netty 3 to netty 4 completely changes that proposition.

For these reasons I am against this, unless there is a stronger argument that isn't "users cannot wait for a few more months" (I also haven't heard of anyone complaining about netty4 not being available right now).

At the very least this should be discussed on the mailing list.

M1 releases are useful for testing but of no use to production users,

@mdedetrich
Copy link
Contributor

mdedetrich commented Nov 5, 2023

M1 releases are useful for testing but of no use to production users

Thats besides the point, if production users really want netty4 they can test and then use 1.1.0-M1 themselves otherwise they can wait for an actual 1.1.0 release (milestones need to be voted on anyways, just like with any other release so its going to be just as tested and those production users can even help out in testing that M1 release!).

I really do not see how tainting the entire 1.0.x branch which is explicitly against the entire point of the branch is worth it for this.

@pjfanning
Copy link
Member Author

I would be very disappointed to see users putting a M1 release into production use. Having a 1.1.0 release that is not LTS but where we promise to follow up with an LTS 1.2.0 - I think this would be better. In the absence of any consensus, then this change is low risk - it adds netty 4 support but this is opt-in and the changes do not appear to make it harder for netty 3 users.

@mdedetrich
Copy link
Contributor

mdedetrich commented Nov 5, 2023

I would be very disappointed to see users putting a M1 release into production use.

Why? Its just as tested as any other Pekko release, seeing as it has to go through the entire ASF release process. Furthermore, its actually a good thing to make an 1.1.0-M1 that is tested before an actual release of 1.1.0 because it means we get validated real world usage before the full release.

I mean this is the whole essence behind a milestone.

In the absence of any consensus, then this change is low risk - it adds netty 4 support but this is opt-in and the changes do not appear to make it harder for netty 3 users.

It doesn't matter if the change is low risk, by definition the change from netty3 to netty4 is massive and there are significant changes there (as you can see in the PR's themselves). The entire point of Pekko 1.0.x is that it is essentially Akka 2.6.20 + necessary renames + migration features + any critital fixes/security issues, thats it and nothing more. This was an agreement that was made after many months of discussion by the community and its not fair to upend that, at least without proper consensus (and that does not mean lack of one)

@He-Pin
Copy link
Member

He-Pin commented Nov 10, 2023

I think we should only ship it in 1.1.x and we can release M1 in this month? I expect a release of 1.1.0 in before the end of 2023.

@mdedetrich
Copy link
Contributor

I think we should only ship it in 1.1.x and we can release M1 in this month? I expect a release of 1.1.0 in before the end of 2023.

I am expecting full release of 1.1.0 early next year but yes lets see, the inliner changes still haven't been reviewed yet and there may need to be more work there and remember that December is christmas period for us.

M1 should be released after the slf4j 2 changes land in #784

@He-Pin
Copy link
Member

He-Pin commented Dec 24, 2023

@pjfanning as you plan to add a new milestone release for 1.1.x does this still be valid?

@pjfanning
Copy link
Member Author

I'll close this.

@pjfanning pjfanning closed this Dec 24, 2023
@pjfanning pjfanning deleted the netty4-support branch October 9, 2025 20:01
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.

3 participants