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

Remove rolling upgrade serialization of ORSet[ActorRef] #25336

Closed
patriknw opened this Issue Jul 10, 2018 · 3 comments

Comments

Projects
None yet
1 participant
@patriknw
Member

patriknw commented Jul 10, 2018

Revert the double serialization that was added in
b72312d to support rolling upgrades.
Explained in PR #25208

@patriknw patriknw self-assigned this Jul 10, 2018

@patriknw patriknw added this to the 2.5.15 milestone Jul 10, 2018

@patriknw

This comment has been minimized.

Show comment
Hide comment
@patriknw

patriknw Aug 24, 2018

Member

Let's keep this in 2.5.15 also and remove in next

Member

patriknw commented Aug 24, 2018

Let's keep this in 2.5.15 also and remove in next

@patriknw patriknw modified the milestones: 2.5.15, 2.5.16, 2.5.17 Aug 24, 2018

@patriknw

This comment has been minimized.

Show comment
Hide comment
@patriknw

patriknw Aug 29, 2018

Member

also kept in 2.5.16 because that was a quick security patch

Member

patriknw commented Aug 29, 2018

also kept in 2.5.16 because that was a quick security patch

patriknw added a commit that referenced this issue Sep 11, 2018

patriknw added a commit that referenced this issue Sep 11, 2018

patriknw added a commit that referenced this issue Sep 11, 2018

patriknw added a commit that referenced this issue Sep 26, 2018

Merge pull request #25595 from akka/wip-25336-orset-actorref-patriknw
Revert Support for rolling upgrade of ORSet[ActorRef], #25336

@patriknw patriknw closed this Sep 26, 2018

@patriknw

This comment has been minimized.

Show comment
Hide comment
@patriknw

patriknw Sep 26, 2018

Member

Note that this was not a revert due to a regression. It was a planned two-phase change to be able to support rolling updates.

Member

patriknw commented Sep 26, 2018

Note that this was not a revert due to a regression. It was a planned two-phase change to be able to support rolling updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment