You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Zoltan Chovan (Code Review)" <ge...@cloudera.org> on 2021/11/29 17:17:28 UTC

[kudu-CR] start kudu.sh script changes

Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18049

to look at the new patch set (#3).

Change subject: start_kudu.sh script changes
......................................................................

start_kudu.sh script changes

- removed the readlink -f line as it wasn't providing any value
- added check for detecting if $KUDU_HOME is set or $KUDU_HOME/www is
available
- WEBSERVER_DOC_ROOT was made optional, doesn't stop the startup if not
  found
- webserver_doc_root flag is not set on master/tserver if not found
- added warning to clarify the usage of -b|--builddir flag

Change-Id: I0057a6d6668488001be8b4f74f46010e2620b46e
---
M src/kudu/scripts/start_kudu.sh
1 file changed, 22 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/18049/3
-- 
To view, visit http://gerrit.cloudera.org:8080/18049
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0057a6d6668488001be8b4f74f46010e2620b46e
Gerrit-Change-Number: 18049
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>