Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15641 )
Change subject: IMPALA-9097: Don't require minicluster for backend tests ...................................................................... IMPALA-9097: Don't require minicluster for backend tests Currently, many backend tests require a running minicluster, because they initialize a Frontend object that requires a connection to the Hive Metastore. If the minicluster is not running or if cluster configurations are missing (i.e. bin/create-test-configurations.sh needs to run), the backend tests will fail. The docker based tests always hit this, because they run the backend tests without a minicluster. The HMS dependency comes from the Frontend's MetaStoreClientPool, which is unnecesary for backend tests. This modifies the code so that it does not initialize this for backend tests, and thus backend tests pass without a running minicluster. Testing: - Ran backend tests without a running minicluster Change-Id: I8f1b1385853fb23df28d24d38761237e6e5c97a7 Reviewed-on: http://gerrit.cloudera.org:8080/15641 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/service/frontend.cc M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/main/java/org/apache/impala/service/JniFrontend.java 3 files changed, 24 insertions(+), 12 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/15641 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8f1b1385853fb23df28d24d38761237e6e5c97a7 Gerrit-Change-Number: 15641 Gerrit-PatchSet: 5 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
