Thomas Tauber-Marshall has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15778


Change subject: Ensure the FE custom cluster tests are run in precommit
......................................................................

Ensure the FE custom cluster tests are run in precommit

Prior to this patch, in run-all-tests.sh it was required that both
CLUSTER_TEST=true and FE_TEST=true for the FE custom cluster tests to
get run.

This resulted in these tests not getting run in precommit after the
precommit job was split into two runs of separate tests, one with
FE_TEST=true and the other with CLUSTER_TEST=true.

Its probably better to maintain the invariant that all of the tests
can be run by setting each of FE_TEST, BE_TEST, etc. to true one at a
time, so this patch fixes the issue by removing the requirement that
FE_TEST=true for the FE custom cluster tests to be run.

This doesn't introduce any new dependencies since having the FE built
is a requirement for being able to run the non-FE custom cluster tests
anyways.

Change-Id: I111a7993ca6c2f986b496c489f5cf028ee0d5154
---
M bin/run-all-tests.sh
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/15778/1
--
To view, visit http://gerrit.cloudera.org:8080/15778
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I111a7993ca6c2f986b496c489f5cf028ee0d5154
Gerrit-Change-Number: 15778
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>

Reply via email to