Skip to content

ZOOKEEPER-2864: Add script to run a java api compatibility tool#329

Closed
afine wants to merge 1 commit intoapache:branch-3.5from
afine:ZOOKEEPER-2864
Closed

ZOOKEEPER-2864: Add script to run a java api compatibility tool#329
afine wants to merge 1 commit intoapache:branch-3.5from
afine:ZOOKEEPER-2864

Conversation

@afine
Copy link
Copy Markdown
Contributor

@afine afine commented Aug 7, 2017

For example: python check_compatibility.py branch-3.5..branch-3.4

@hanm
Copy link
Copy Markdown
Contributor

hanm commented Aug 9, 2017

I suggest we put this script under zookeeper/src/java/test/bin/ where other scripts are currently located, so it's consistent.

A better solution is to consolidate all scripts and put them in a folder with name making more sense, like other projects (e.g. hbase who puts scripts under root/dev-support), but that should be done separately as moving scripts will break a couple of work flows and require coordination.

@afine
Copy link
Copy Markdown
Contributor Author

afine commented Aug 10, 2017

@hanm I moved the script as you suggested.

Copy link
Copy Markdown
Contributor

@hanm hanm left a comment

Choose a reason for hiding this comment

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

LGTM.

Document this on cwiki and make it part of release auditing process will be sweet.

asfgit pushed a commit that referenced this pull request Aug 10, 2017
For example: `python check_compatibility.py branch-3.5..branch-3.4 `

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes #329 from afine/ZOOKEEPER-2864
asfgit pushed a commit that referenced this pull request Aug 10, 2017
For example: `python check_compatibility.py branch-3.5..branch-3.4 `

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes #329 from afine/ZOOKEEPER-2864

(cherry picked from commit 82be7f6)
Signed-off-by: Michael Han <hanm@apache.org>
@asfgit asfgit closed this in 3408b62 Aug 10, 2017
@hanm
Copy link
Copy Markdown
Contributor

hanm commented Aug 10, 2017

Committed to master: 3408b62
3.5: 82be7f6
3.4: 4beaa69

lvfangmin pushed a commit to lvfangmin/zookeeper that referenced this pull request Jun 17, 2018
For example: `python check_compatibility.py branch-3.5..branch-3.4 `

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes apache#329 from afine/ZOOKEEPER-2864

(cherry picked from commit 82be7f6)
Signed-off-by: Michael Han <hanm@apache.org>
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Sep 3, 2022
For example: `python check_compatibility.py branch-3.5..branch-3.4 `

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes apache#329 from afine/ZOOKEEPER-2864

(cherry picked from commit 82be7f6)
Signed-off-by: Michael Han <hanm@apache.org>
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