Matthew Jacobs has posted comments on this change.

Change subject: IMPALA-3417: run-all.sh fails when no services should start
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/3250/1/testdata/cluster/admin
File testdata/cluster/admin:

Line 320: 
> remove?
Done


Line 321:   for SERVICE in ${SUPPORTED_SERVICES[@]-}; do
> how about moving this into the if block?
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I268c7ad66c82f2b04b832d520e21662e631572cf
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-HasComments: Yes

Reply via email to