You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Dan Hecht (Code Review)" <ge...@cloudera.org> on 2016/02/26 08:59:03 UTC

[Impala-CR](cdh5-trunk) Get and use Kudu from the toolchain by default

Dan Hecht has posted comments on this change.

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


Patch Set 3: Code-Review+2

(3 comments)

Please get a review from a kudu person too, especially regarding the testdata stuff.

http://gerrit.cloudera.org:8080/#/c/1985/3/bin/impala-config.sh
File bin/impala-config.sh:

Line 92: cammond
typo


Line 222: #       before release this needs to be updated to the version about to be released.
how will we remember to do this?


Line 223: export KUDU_JAVA_VERSION=0.6.0
where is this used?


-- 
To view, visit http://gerrit.cloudera.org:8080/1985
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3db88cbd27f2ea2394f011bc8d1face37411ed58
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <ca...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Casey Ching <ca...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@cloudera.com>
Gerrit-HasComments: Yes