Adar Dembo has posted comments on this change.

Change subject: build: run local tests parallel with distributed tests
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/1815/3/build-support/jenkins/build-and-test.sh
File build-support/jenkins/build-and-test.sh:

Line 307:   pushd $SOURCE_ROOT/python
> Should probably popd this after running the tests.
I understand skipping the rest, but could you at least add the popd here? 
Strictly speaking we didn't need it before, since the Python tests were last to 
run in the script, but since you've moved them, that's no longer the case.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaac391210589a68b43900deb28abe9aff2fe5c76
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to