Casey Ching has posted comments on this change.

Change subject: Minor Kudu build fixes
......................................................................


Patch Set 2:

> > > 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

Ah, got it.

-- 
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: 2
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

Reply via email to