mdedetrich commented on PR #587: URL: https://github.com/apache/incubator-pekko/pull/587#issuecomment-1702276993
@pjfanning I have been thinking about this and I do believe that some version of these changes need to be put into Pekko 1.0.x. This is due to the fact that Pekko 1.0.x primary goal is acting as a bridge from Akka to Pekko 1.1.x and so if we merge this PR only into 1.1.x we can unintentionally break users. I also am not sure how much of a problem it would be if such a change would be put into the Pekko 1.0.x branch. As far as I understand this is for protocol changes for the management of the cluster, not the cluster itself which means that you should be able to easily update from a Pekko 1.0.0/1.0.1 cluster to a Pekko cluster with this fix without any real issue (someone needs to comment on this). If this is not the case then this PR for Pekko 1.1.x is fine but we would still need to add code into Pekko 1.0.x to provide a migration path in addition to changing the manager name. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
