Attila Bukor has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18049 )

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
Reviewed-on: http://gerrit.cloudera.org:8080/18049
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
Reviewed-by: Attila Bukor <[email protected]>
---
M src/kudu/scripts/start_kudu.sh
1 file changed, 22 insertions(+), 6 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved

-- 
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: merged
Gerrit-Change-Id: I0057a6d6668488001be8b4f74f46010e2620b46e
Gerrit-Change-Number: 18049
Gerrit-PatchSet: 4
Gerrit-Owner: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to