Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14535 )
Change subject: scripts: multi-master support for start_kudu.sh and stop_kudu.sh ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/14535/1/src/kudu/scripts/start_kudu.sh File src/kudu/scripts/start_kudu.sh: http://gerrit.cloudera.org:8080/#/c/14535/1/src/kudu/scripts/start_kudu.sh@31 PS1, Line 31: -h, --help Print help : -m, --num-masters Number of Kudu Master servers to start (default: 1) : -t, --num-tservers Number of Kudu Tablet Servers to start (default: 3) : --rpc-master RPC port of first Kudu Master; HTTP port is the next number. : Subsequent Masters will have following numbers : --rpc-tserver RPC port of first Kudu Tablet Server; HTTP port is the next : number. Subsequent Tablet Serv > nit: align the description of the parameters? Done http://gerrit.cloudera.org:8080/#/c/14535/1/src/kudu/scripts/stop_kudu.sh File src/kudu/scripts/stop_kudu.sh: http://gerrit.cloudera.org:8080/#/c/14535/1/src/kudu/scripts/stop_kudu.sh@25 PS1, Line 25: pkill -9 -e -u $(id -u) -x kudu-tserver : pkill -9 -e -u $(id -u) -x kudu-master : : : > If relying on pgrep, pkill comes along as well. Maybe, simply do Done http://gerrit.cloudera.org:8080/#/c/14535/1/src/kudu/scripts/stop_kudu.sh@30 PS1, Line 30: : : : : > ditto: Done -- To view, visit http://gerrit.cloudera.org:8080/14535 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id8db19c9145b5c68bb437390e6c0608961680347 Gerrit-Change-Number: 14535 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Wed, 23 Oct 2019 22:11:54 +0000 Gerrit-HasComments: Yes
