Dan Hecht has posted comments on this change. Change subject: Minor Kudu build fixes ......................................................................
Patch Set 1: > > Doesn't boostrap_toolchain.sh also need to be updated? When i > > tried just that impala-config.sh change, it still tried to > download > > the package. > > There is nothing to download because Ubuntu 12 isn’t supported. I > just double checked > http://archive.cloudera.com/beta/impala-kudu/parcels/latest/ > > You resourced impala-config.sh? If so can you print out > "$DISTRO_VERSION" and tell me what it is. What I mean is that bootstrap_toolchain.sh doesn't seem to care about whether KUDU_IS_SUPPORTED or not, so it still tries to download the package even though it won't exist. dhecht@linux:~/src/Impala$ echo $DISTRO_VERSION ubuntu12 -- To view, visit http://gerrit.cloudera.org:8080/2547 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6412ea0c79c9f2a2e3285b532372076ca437400d Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Casey Ching <[email protected]> Gerrit-Reviewer: Casey Ching <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-HasComments: No
