Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15415 )

Change subject: [ranger] parameterize the Java binary path for the subprocess
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15415/1/src/kudu/ranger/ranger_client.cc
File src/kudu/ranger/ranger_client.cc:

http://gerrit.cloudera.org:8080/#/c/15415/1/src/kudu/ranger/ranger_client.cc@145
PS1, Line 145:
> Since we use execvp() call to start the subprocess, based on man for execvp
Yeah but we BaseName argv[0], which seems to confuse things. Maybe this is the 
case only on Mac, or there was something weird in my env with postgres, but it 
was finding the wrong postgres for me using the same method. Can you double 
check what happens if there's another java in the path? Maybe run "java 
-version" using the same method.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic8f23a4e96ec5fd29de0e6d598d4400f5ac92366
Gerrit-Change-Number: 15415
Gerrit-PatchSet: 7
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 18 Mar 2020 00:21:30 +0000
Gerrit-HasComments: Yes

Reply via email to