Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13004 )

Change subject: [build] Fix the java compatibility checker script
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13004/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/13004/1//COMMIT_MSG@13
PS1, Line 13:
> Nit: extra space here.
Done


http://gerrit.cloudera.org:8080/#/c/13004/1/build-support/check_compatibility.py
File build-support/check_compatibility.py:

http://gerrit.cloudera.org:8080/#/c/13004/1/build-support/check_compatibility.py@204
PS1, Line 204:   # Run the build in each tree.
> Hmm, would be nice if this would work on existing build output, since it'll
I agree that would be a good enhancement for follow on changes. For now, given 
the gradle build has been around a while, this is better than nothing and can 
be use for releases and arbitrary commits.



--
To view, visit http://gerrit.cloudera.org:8080/13004
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id482da0b3aa5ea35fcfb2b674f0d7ae413045ca3
Gerrit-Change-Number: 13004
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Adar Lieber-Dembo <a...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 07 Jan 2021 21:20:53 +0000
Gerrit-HasComments: Yes

Reply via email to