Andrew Wong 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/15438/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15438/3//COMMIT_MSG@10 PS3, Line 10: So that it can be found even with the default value of gflag : as 'ranger_jar_path'. The same for the hms JAR. > I think the concern is the link can be broken when the master is running. But we only use the JAR link once -- when we initialize the master -- so it shouldn't matter if the link gets broken while the master is running, right? My point about the default changing is that this doesn't seem to "fix" some broken behavior, so I'm trying to better understand the intent behind this patch. -- 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: 3 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 07:09:31 +0000 Gerrit-HasComments: Yes
