----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51755/#review153098 -----------------------------------------------------------
itests/src/test/resources/testconfiguration.properties (line 53) <https://reviews.apache.org/r/51755/#comment222376> Lets do it other way round. Write all new tests on MiniLlapLocalDriver and have one on MR. ql/src/java/org/apache/hadoop/hive/ql/optimizer/SimpleFetchOptimizer.java (lines 130 - 132) <https://reviews.apache.org/r/51755/#comment222377> Is this needed? ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSortLimitPullUpConstantsRule.java (lines 162 - 170) <https://reviews.apache.org/r/51755/#comment222378> Now this is covered in different jira, can be removed. ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java (line 1528) <https://reviews.apache.org/r/51755/#comment222380> Add opcode in error message. ql/src/test/org/apache/hadoop/hive/ql/parse/TestSQL11ReservedKeyWordsNegative.java (line 41) <https://reviews.apache.org/r/51755/#comment222381> Any reason for adding Minus as reserved word? ql/src/test/results/clientpositive/auto_join8.q.out (line 94) <https://reviews.apache.org/r/51755/#comment222372> Not folding null constant anymore? ql/src/test/results/clientpositive/input25.q.out (line 122) <https://reviews.apache.org/r/51755/#comment222375> Shuffling unnecssary constant column? ql/src/test/results/clientpositive/join8.q.out (line 97) <https://reviews.apache.org/r/51755/#comment222373> Not folding null constant anymore? ql/src/test/results/clientpositive/smb_mapjoin_25.q.out (line 116) <https://reviews.apache.org/r/51755/#comment222374> Shuffling unnecessary column? - Ashutosh Chauhan On Oct. 15, 2016, 4:10 a.m., pengcheng xiong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51755/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2016, 4:10 a.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Repository: hive-git > > > Description > ------- > > HIVE-12765 > > > Diffs > ----- > > itests/src/test/resources/testconfiguration.properties 97e310d > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SimpleFetchOptimizer.java > eb0ba7b > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveCalciteUtil.java > c527e58 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRelFactories.java > cf93ed8 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveExcept.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveIntersect.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveExceptRewriteRule.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveIntersectMergeRule.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveIntersectRewriteRule.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSortLimitPullUpConstantsRule.java > cc318db > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ASTConverter.java > 8d738aa > ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java e6ab947 > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 5d3fa6a > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 5c16c55 > ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 50987c3 > ql/src/java/org/apache/hadoop/hive/ql/parse/QBExpr.java cccf0f6 > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 747f387 > > ql/src/test/org/apache/hadoop/hive/ql/parse/TestSQL11ReservedKeyWordsNegative.java > a427803 > ql/src/test/queries/clientpositive/except_all.q PRE-CREATION > ql/src/test/queries/clientpositive/except_distinct.q PRE-CREATION > ql/src/test/queries/clientpositive/intersect_all.q PRE-CREATION > ql/src/test/queries/clientpositive/intersect_distinct.q PRE-CREATION > ql/src/test/queries/clientpositive/intersect_merge.q PRE-CREATION > ql/src/test/results/clientpositive/auto_join8.q.out ccbafba > ql/src/test/results/clientpositive/except_all.q.out PRE-CREATION > ql/src/test/results/clientpositive/except_distinct.q.out PRE-CREATION > ql/src/test/results/clientpositive/input25.q.out 465173d > ql/src/test/results/clientpositive/intersect_all.q.out PRE-CREATION > ql/src/test/results/clientpositive/intersect_distinct.q.out PRE-CREATION > ql/src/test/results/clientpositive/join8.q.out 47b821a > ql/src/test/results/clientpositive/list_bucket_query_oneskew_2.q.out > 45f2481 > ql/src/test/results/clientpositive/smb_mapjoin_25.q.out aba899e > ql/src/test/results/clientpositive/spark/auto_join8.q.out 6909113 > ql/src/test/results/clientpositive/spark/join8.q.out 4903c90 > ql/src/test/results/clientpositive/spark/smb_mapjoin_25.q.out eacb438 > ql/src/test/results/clientpositive/spark/union_top_level.q.out e1c7fc7 > ql/src/test/results/clientpositive/tez/intersect_merge.q.out PRE-CREATION > ql/src/test/results/clientpositive/timestamp.q.out 7c08ec8 > ql/src/test/results/clientpositive/vector_null_projection.q.out 779f787 > > Diff: https://reviews.apache.org/r/51755/diff/ > > > Testing > ------- > > > Thanks, > > pengcheng xiong > >
