You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by tm...@apache.org on 2018/02/16 19:15:23 UTC

[2/3] impala git commit: Bump Kudu version to 0eef8e0

Bump Kudu version to 0eef8e0

This pulls in support for DECIMAL.

Change-Id: Id983ceb8806b2a002fadb19c065267d45f4c65db
Reviewed-on: http://gerrit.cloudera.org:8080/9348
Reviewed-by: Lars Volker <lv...@cloudera.com>
Reviewed-by: Grant Henke <gr...@gmail.com>
Tested-by: Impala Public Jenkins


Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/be675e15
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/be675e15
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/be675e15

Branch: refs/heads/master
Commit: be675e15578eda6285c9558c21bc16ce9dfb7ffc
Parents: 61f892a
Author: Thomas Tauber-Marshall <tm...@cloudera.com>
Authored: Thu Feb 15 18:29:33 2018 -0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Fri Feb 16 06:36:48 2018 +0000

----------------------------------------------------------------------
 be/src/exec/kudu-util.cc | 3 +++
 bin/impala-config.sh     | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/impala/blob/be675e15/be/src/exec/kudu-util.cc
----------------------------------------------------------------------
diff --git a/be/src/exec/kudu-util.cc b/be/src/exec/kudu-util.cc
index 03cb51f..d32df25 100644
--- a/be/src/exec/kudu-util.cc
+++ b/be/src/exec/kudu-util.cc
@@ -193,6 +193,9 @@ ColumnType KuduDataTypeToColumnType(DataType type) {
     case DataType::DOUBLE: return ColumnType(PrimitiveType::TYPE_DOUBLE);
     case DataType::BINARY: return ColumnType(PrimitiveType::TYPE_BINARY);
     case DataType::UNIXTIME_MICROS: return ColumnType(PrimitiveType::TYPE_TIMESTAMP);
+    case DataType::DECIMAL:
+      DCHECK(false) << "DECIMAL is not supported on Kudu.";
+      return ColumnType(PrimitiveType::INVALID_TYPE);
   }
   return ColumnType(PrimitiveType::INVALID_TYPE);
 }

http://git-wip-us.apache.org/repos/asf/impala/blob/be675e15/bin/impala-config.sh
----------------------------------------------------------------------
diff --git a/bin/impala-config.sh b/bin/impala-config.sh
index bb1cb15..d13e68b 100755
--- a/bin/impala-config.sh
+++ b/bin/impala-config.sh
@@ -72,7 +72,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.
-export IMPALA_TOOLCHAIN_BUILD_ID=39-a1bbd2851a
+export IMPALA_TOOLCHAIN_BUILD_ID=53-d95bb7f778
 # Versions of toolchain dependencies.
 # -----------------------------------
 export IMPALA_AVRO_VERSION=1.7.4-p4
@@ -158,7 +158,7 @@ if [[ $OSTYPE == "darwin"* ]]; then
 fi
 
 # Kudu version in the toolchain; provides libkudu_client.so and minicluster binaries.
-export IMPALA_KUDU_VERSION=1520b39
+export IMPALA_KUDU_VERSION=0eef8e0
 unset IMPALA_KUDU_URL
 
 # Kudu version used to identify Java client jar from maven