-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25743/
-----------------------------------------------------------
(Updated Sept. 18, 2014, 5:40 p.m.)
Review request for hive.
Changes
-------
Avoid some array copies in TezWork and removed the try/catch surrounding the
call to add local resources to the Tez DAG. Added some more unit tests for
TezWork changes.
Bugs: HIVE-7950
https://issues.apache.org/jira/browse/HIVE-7950
Repository: hive-git
Description
-------
Was trying to run some queries using the AccumuloStorageHandler when using the
Tez execution engine. Some things that classes which were added to tmpjars
weren't making it into the container. When a Tez Session is already open, as is
the normal case when simply using the `hive` command, the resources aren't
added.
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java
0d0ac41
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java 428e0ff
ql/src/java/org/apache/hadoop/hive/ql/plan/TezWork.java 456b5eb
ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezSessionPool.java
ad5a6e7
ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezTask.java 45ab672
ql/src/test/org/apache/hadoop/hive/ql/plan/TestTezWork.java 2600149
Diff: https://reviews.apache.org/r/25743/diff/
Testing
-------
Ran ql/ unit tests, tested AccumuloStorageHandler in local deployment after
killing tez session.
Thanks,
Josh Elser
