Fang-Yu Rao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15590
Change subject: IMPALA-9566: Sentry service should not be started after IMPALA-8870 ...................................................................... IMPALA-9566: Sentry service should not be started after IMPALA-8870 Due to the incompatibility between different versions of the Guava libraries, after bumping up Guava in IMPALA-8870, our build script is not supposed to start up the Sentry service when starting the minicluster because Sentry has not had its Guava bumped up yet. However, the patch for IMPALA-8870 did not take this into consideration when $TARGET_FILESYSTEM is s3 and thus run-all.sh still attempts to start up Sentry in this case. This patch fixes the bug. Testing: - Verified that this patch passes the core tests in the DEBUG build when $TARGET_FILESYSTEM is s3. Change-Id: If81846f4251fb2aa752ba8c33615cae0ab513a62 --- M testdata/bin/run-all.sh 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/90/15590/2 -- To view, visit http://gerrit.cloudera.org:8080/15590 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: If81846f4251fb2aa752ba8c33615cae0ab513a62 Gerrit-Change-Number: 15590 Gerrit-PatchSet: 2 Gerrit-Owner: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]>
