Adar Dembo has posted comments on this change. Change subject: [java] Remove double brackets from wrapper script ......................................................................
Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7655/2//COMMIT_MSG Commit Message: PS2, Line 12: This is because the wrapper script is a shell script : and not a bash script FWIW, we do enforce that /bin/bash is the interpreter for all Kudu shell scripts. We can do that for gradlew too, if that's interesting (though if it's just copied into our repo verbatim from an external source, I can also understand keeping it this way). -- To view, visit http://gerrit.cloudera.org:8080/7655 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I839c47bbef8bf901047b9379be958f4cebbd406e Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes