-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55994/
-----------------------------------------------------------
(Updated Jan. 26, 2017, 8:09 p.m.)
Review request for hive, Mohit Sabharwal, Sahil Takiar, and Vihang Karajgaonkar.
Bugs: HIVE-15736
https://issues.apache.org/jira/browse/HIVE-15736
Repository: hive-git
Description (updated)
-------
Added unit tests on TestUtilities to validate
- Single and multiple threads
- InputEstimator usage
- ContentSummaryInputFormat usage.
It also fixed an issue with the InputEstimator scenario where the values
returned by the InputEstimator where overriden later by the correct filesystem
calls.
An interesting thing (code commented while it is on review) is that when
executing the InputEstimator code path the line commented seems are not needed.
It might
be that the idea was to set some configurations to the jobConf, but the jobConf
was never passed as parameter to the estimate method. Please help me verify
this.
Diffs
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java
68dd5e7247415dec1e353010ea34481c4f2fc6cd
ql/src/test/org/apache/hadoop/hive/ql/exec/InputEstimatorTestClass.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/exec/TestUtilities.java
e444946e990d9adb90ce24837cfe4edcf5126d3a
Diff: https://reviews.apache.org/r/55994/diff/
Testing
-------
Waiting for tests HiveQA
Thanks,
Sergio Pena
