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

ZOOKEEPER-3780: restore Version.getRevision() to be bacward compatible #1309

Closed
wants to merge 1 commit into from

Conversation

nkalmar
Copy link
Contributor

@nkalmar nkalmar commented Apr 7, 2020

for branch-3.5

@nkalmar
Copy link
Contributor Author

nkalmar commented Apr 8, 2020

retest maven build

@symat
Copy link
Contributor

symat commented Apr 9, 2020

retest ant build

Copy link
Contributor

@symat symat left a comment

Choose a reason for hiding this comment

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

+1
looks identical to the change for branch-3.6 / master

@symat
Copy link
Contributor

symat commented Apr 9, 2020

@nkalmar I don't see how is this different from #1304
cherrypick to branch-3.5 wouldn't be easier?
maybe I just miss something...

@nkalmar
Copy link
Contributor Author

nkalmar commented Apr 9, 2020

It is the same, I thought @eolivelli said it didn't pick clean? Please just cherry pick #1304 if it picks clean. Thanks!

@symat
Copy link
Contributor

symat commented Apr 9, 2020

OK, I'll try it. Thanks!

@nkalmar
Copy link
Contributor Author

nkalmar commented Apr 27, 2020

I'll merge this as 3.5 release is coming up, we don't wan't to miss it. (Already merged in master and 3.6)

asfgit pushed a commit that referenced this pull request Apr 27, 2020
for branch-3.5

Author: Norbert Kalmar <nkalmar@apache.org>

Reviewers: Mate Szalay-Beko <symat@apache.org>

Closes #1309 from nkalmar/branch-3.5
@nkalmar nkalmar closed this Apr 27, 2020
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