You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by mj...@apache.org on 2016/12/12 22:36:38 UTC
incubator-impala git commit: IMPALA-4642: Revert Kudu to unblock gvos
Repository: incubator-impala
Updated Branches:
refs/heads/master 39017adff -> 2414d4c9c
IMPALA-4642: Revert Kudu to unblock gvos
This reverts commit 39017adffc0cb13ebbdab89902d8403faf9a2b4b.
Bumping the Kudu version seems to have exposed a bug where
fragments aren't torn down even after the query completes
(see IMPALA-4645). That breaks TestFragmentLifecycle in all
test runs. Reverting the Kudu commit allows the tests to
complete normally, though the underlying issue still needs
to be investigated (IMPALA-4645).
I verified that the TestFragmentLifecycle tests now pass with
this commit.
Change-Id: I94680aa45c694cca4feff7c3aa94da837174a004
Reviewed-on: http://gerrit.cloudera.org:8080/5470
Reviewed-by: Alex Behm <al...@cloudera.com>
Tested-by: Matthew Jacobs <mj...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/2414d4c9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/2414d4c9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/2414d4c9
Branch: refs/heads/master
Commit: 2414d4c9c6f48e86629a87d454dc4ecf8691267d
Parents: 39017ad
Author: Matthew Jacobs <mj...@cloudera.com>
Authored: Sun Dec 11 09:38:45 2016 -0800
Committer: Matthew Jacobs <mj...@cloudera.com>
Committed: Sun Dec 11 22:19:34 2016 +0000
----------------------------------------------------------------------
bin/impala-config.sh | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/2414d4c9/bin/impala-config.sh
----------------------------------------------------------------------
diff --git a/bin/impala-config.sh b/bin/impala-config.sh
index 6ce062b..80ce7d6 100755
--- a/bin/impala-config.sh
+++ b/bin/impala-config.sh
@@ -57,7 +57,7 @@ fi
# moving to a different build of the toolchain, e.g. when a version is bumped or a
# compile option is changed. The build id can be found in the output of the toolchain
# build jobs, it is constructed from the build number and toolchain git hash prefix.
-: ${IMPALA_TOOLCHAIN_BUILD_ID=308-96a4cc516e}
+: ${IMPALA_TOOLCHAIN_BUILD_ID=304-a5cb8de41a}
# This flag is used in $IMPALA_HOME/cmake_modules/toolchain.cmake.
# If it's 0, Impala will be built with the compiler in the toolchain directory.
@@ -270,6 +270,7 @@ export IMPALA_GFLAGS_VERSION=2.0
export IMPALA_GLOG_VERSION=0.3.2-p2
export IMPALA_GPERFTOOLS_VERSION=2.5
export IMPALA_GTEST_VERSION=1.6.0
+export IMPALA_KUDU_VERSION=a70c905006
export IMPALA_LLVM_VERSION=3.8.0-p1
export IMPALA_LLVM_ASAN_VERSION=3.8.0-p1
# Debug builds should use the release+asserts build to get additional coverage.
@@ -290,14 +291,12 @@ export IMPALA_THRIFT_VERSION=0.9.0-p8
export IMPALA_THRIFT_JAVA_VERSION=0.9.0
export IMPALA_ZLIB_VERSION=1.2.8
-# Kudu version in the toolchain; provides libkudu_client.so and minicluster binaries.
-export IMPALA_KUDU_VERSION=e018a83
-
-# Kudu version used to identify Java client jar from maven
-export KUDU_JAVA_VERSION=1.2.0-SNAPSHOT
-
export KUDU_MASTER="${KUDU_MASTER:-127.0.0.1}"
export KUDU_MASTER_PORT="${KUDU_MASTER_PORT:-7051}"
+# TODO: Figure out a way to use a snapshot version without causing a lot of breakage due
+# to nightly changes from Kudu. The version below is the last released version but
+# before release this needs to be updated to the version about to be released.
+export KUDU_JAVA_VERSION=1.2.0-SNAPSHOT
if [[ $OSTYPE == "darwin"* ]]; then
IMPALA_CYRUS_SASL_VERSION=2.1.26