----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24909/#review51186 -----------------------------------------------------------
Ship it! Ship It! - Brock Noland On Aug. 20, 2014, 8:37 p.m., Venki Korukanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24909/ > ----------------------------------------------------------- > > (Updated Aug. 20, 2014, 8:37 p.m.) > > > Review request for hive and Thejas Nair. > > > Bugs: HIVE-7807 > https://issues.apache.org/jira/browse/HIVE-7807 > > > Repository: hive-git > > > Description > ------- > > Refer to JIRA HIVE-7807 and HIVE-7001 for details. > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/TestUtilitiesDfs.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 1d6a93a > ql/src/test/org/apache/hadoop/hive/ql/exec/TestUtilities.java bf3fd88 > > Diff: https://reviews.apache.org/r/24909/diff/ > > > Testing > ------- > > Note: I had to create a separate test file for using MiniDFS in itests > module. Existing TestUtilities in ql needs hadoop-test and few other > dependencies which are not needed so far. So decided not to add extra test > dependencies in ql/pom.xml for just one test. > > > Thanks, > > Venki Korukanti > >
