Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive, mail # dev - Review Request 19789: HIVE-6739 Hive HBase query fails on Tez due to missing jars and then due to NPE in getSplits


Copy link to this message
-
Re: Review Request 19789: HIVE-6739 Hive HBase query fails on Tez due to missing jars and then due to NPE in getSplits
Vikram Dixit Kumaraswamy 2014-04-04, 07:53

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19789/#review39521

ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java
<https://reviews.apache.org/r/19789/#comment71934>

    Document all the input params/just rename the params to reflect their use. hdfsPathStr is the destination directory right?

ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java
<https://reviews.apache.org/r/19789/#comment71932>

    Actually looks like this method localizes an array of files.

ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
<https://reviews.apache.org/r/19789/#comment71933>

    Definitely needs method(s) renaming.

ql/src/java/org/apache/hadoop/hive/ql/plan/MapWork.java
<https://reviews.apache.org/r/19789/#comment71936>

    Open a jira for this.

ql/src/java/org/apache/hadoop/hive/ql/plan/TezWork.java
<https://reviews.apache.org/r/19789/#comment71938>

    Aren't you walking the work graph here to populate the job conf?
- Vikram Dixit Kumaraswamy
On April 3, 2014, 2:26 a.m., Sergey Shelukhin wrote: