Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 )
Change subject: [subprocess] copy subprocess JAR to be next to the master binary ...................................................................... Patch Set 2: > Patch Set 1: > > I think we can remove this too : > https://github.com/apache/kudu/blob/master/build-support/dist_test.py#L113-L117 Hmm, it seems we have to keep this for the test to know where to find the jars. -- To view, visit http://gerrit.cloudera.org:8080/15438 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0bab628fa2d1fbb8d9edee32b480c30da2756cdc Gerrit-Change-Number: 15438 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sun, 15 Mar 2020 01:35:02 +0000 Gerrit-HasComments: No
