Tim Armstrong has posted comments on this change.

Change subject: IMPALA-4277: allow overriding of Hive/Hadoop versions/locations
......................................................................


Patch Set 1:

This is for master, yet.

I think we will need to be able to point to the specific directories regardless 
to integrate with other build systems and let people build against arbitrary 
versions of dependencies.

The problem is that the thirdparty Hadoop/Hive/etc tarballs we download have a 
very specific directory layout that is not necessarily shared by other 
source/build trees we might want to build against. E.g. it would make sense to 
build against the Hive source directly, but in the Hive source repo 
hive_metastore.thrift is under ./metastore/if/hive_metastore.thrift instead of 
./src/metastore/if/hive_metastore.thrift

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7da10e38f9c4309f2d193dc25f14a6ea308c9639
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-HasComments: No

Reply via email to