----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16229/#review30486 -----------------------------------------------------------
ant/src/org/apache/hadoop/hive/ant/QTestGenTask.java <https://reviews.apache.org/r/16229/#comment58320> that is because the new test has no result directory, it compares outputs of multiple queries ant/src/org/apache/hadoop/hive/ant/QTestGenTask.java <https://reviews.apache.org/r/16229/#comment58321> yeah, it ends up being written to build output. Do you think it should be written elsewhere? ql/src/test/queries/clientcompare/vectorized_math_funcs_00.qv <https://reviews.apache.org/r/16229/#comment58355> this will muddy the naming convention for versions and potentially cause ambiguity; discussed the details in person - Sergey Shelukhin On Dec. 13, 2013, 7:15 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16229/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2013, 7:15 p.m.) > > > Review request for hive and Jitendra Pandey. > > > Bugs: HIVE-6010 > https://issues.apache.org/jira/browse/HIVE-6010 > > > Repository: hive-git > > > Description > ------- > > See jira. > > > Diffs > ----- > > ant/src/org/apache/hadoop/hive/ant/QTestGenTask.java 85351aa > itests/qtest/pom.xml 8c249a0 > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java c16e82d > ql/src/test/queries/clientcompare/vectorized_math_funcs.q PRE-CREATION > ql/src/test/queries/clientcompare/vectorized_math_funcs_00.qv PRE-CREATION > ql/src/test/queries/clientcompare/vectorized_math_funcs_01.qv PRE-CREATION > ql/src/test/templates/TestCompareCliDriver.vm PRE-CREATION > > Diff: https://reviews.apache.org/r/16229/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
