impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Ribeiro Alves (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) Get and use Kudu from the toolchain by default
Date Tue, 15 Mar 2016 04:26:56 GMT
Hello Todd Lipcon, Dan Hecht,

I'd like you to reexamine a change.  Please visit

to look at the new patch set (#7).

Change subject: Get and use Kudu from the toolchain by default

Get and use Kudu from the toolchain by default

This is for review purposes only. This patch will be merged with David's
big merge patch.

1) Make Kudu compilation dependent on the OS since not all OSs support
2) Only run Kudu related tests when Kudu is supported (see #1).
3) Look for Kudu locally, but in a different location. To use a local
   build of Kudu, set KUDU_BUILD_DIR to the path Kudu was built in and
   set KUDU_CLIENT_DIR to the path KUDU was installed in.
     git clone 3rd party etc...
     mkdir -p $KUDU_BUILD_DIR
     cmake <path to Kudu source dir>
     DESTDIR=$KUDU_CLIENT_DIR make install
4) Look for Kudu in the toolchain if not using a local Kudu build.
5) Add Kudu service startup scripts. The Kudu in the toolchain is
   actually a parcel that has been renamed (the contents were not
   modified in any way), that mean the Kudu service binaries are there.
   Those binaries are now used to run the Kudu service.

Change-Id: I3db88cbd27f2ea2394f011bc8d1face37411ed58
M CMakeLists.txt
M be/CMakeLists.txt
M be/src/exec/CMakeLists.txt
M be/src/exec/
M be/src/exec/
M be/src/exec/kudu-scan-node.h
M be/src/exec/
M be/src/exec/
M be/src/exec/
M be/src/exec/kudu-table-sink.h
M be/src/exec/
M be/src/exec/kudu-util.h
M be/src/runtime/
M bin/
M bin/
M bin/
M common/thrift/
M fe/src/test/java/com/cloudera/impala/analysis/
M fe/src/test/java/com/cloudera/impala/analysis/
M fe/src/test/java/com/cloudera/impala/analysis/
M fe/src/test/java/com/cloudera/impala/analysis/
M fe/src/test/java/com/cloudera/impala/planner/
M fe/src/test/java/com/cloudera/impala/testutil/
M testdata/bin/
M testdata/bin/
D testdata/bin/
D testdata/bin/
M testdata/cluster/admin
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-master
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-tserver
A testdata/cluster/node_templates/cdh5/etc/kudu/master.conf.tmpl
A testdata/cluster/node_templates/cdh5/etc/kudu/tserver.conf.tmpl
M testdata/cluster/node_templates/common/etc/init.d/common.tmpl
M tests/common/
M tests/common/
M tests/metadata/
M tests/metadata/
M tests/query_test/
39 files changed, 347 insertions(+), 104 deletions(-)

  git pull ssh:// refs/changes/85/1985/7
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3db88cbd27f2ea2394f011bc8d1face37411ed58
Gerrit-PatchSet: 7
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <>
Gerrit-Reviewer: Adar Dembo <>
Gerrit-Reviewer: Casey Ching <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: David Ribeiro Alves <>
Gerrit-Reviewer: Todd Lipcon <>

View raw message