Github user evans-ye commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/394#discussion_r218323169
--- Diff: bigtop.bom ---
@@ -110,14 +110,13 @@ bigtop {
*/
dependencies = [
zookeeper:['hadoop', 'hbase'],
- hadoop:['ignite-hadoop', 'hbase', 'crunch', 'pig', 'hive', 'tez',
'sqoop', 'sqoop2',
+ hadoop:['ignite-hadoop', 'hbase', 'crunch', 'hive', 'tez', 'sqoop',
'sqoop2',
'oozie', 'mahout', 'flume', 'giraph', 'solr', 'spark','spark1',
'phoenix', 'alluxio', 'kafka', 'ycsb', 'hama', 'zeppelin',
'tajo', 'apex'
],
hbase:['phoenix','giraph','ycsb','hive'],
- pig:['datafu', 'oozie'],
- hive:['oozie', 'pig','zeppelin'],
+ hive:['oozie', 'zeppelin'],
--- End diff --
Good point. So where does this patch go? If it's going to commit in master,
then your patch works fine. If it's going to commit in branch 1.3, then we need
another patch for 1.3. I expect this should be in 1.3 :)
---
