> On Jan. 28, 2017, 1:30 a.m., Sahil Takiar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java, line 894 > > <https://reviews.apache.org/r/55994/diff/2/?file=1617925#file1617925line894> > > > > Not sure I understand this part. Passing in false invokes > > `Hive.get(doRegisterAllFns=false)`, why does `doRegisterAllFns` need to be > > `false`.
I set it to false because the unit-tests were failing due to HMS starting issues. Instead of fixing the test, I found out the issue was when attempting to register the functions when getting the HMS. Because this line only needs the configuration object from the HMS, then I though it was harmless to set it to false. But, I just thought of a problem when the HMS instance is stored in the cache (with no functions loaded) might cause problems later because functions were not loaded. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55994/#review163375 ----------------------------------------------------------- On Jan. 27, 2017, 7:21 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55994/ > ----------------------------------------------------------- > > (Updated Jan. 27, 2017, 7:21 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 > ------- > > 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/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java > b2c586507483eb5588e07a9ceba3caf395b4d607 > 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 > >