----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72511/#review220807 -----------------------------------------------------------
parser/src/java/org/apache/hadoop/hive/ql/parse/ParseDriverContext.java Lines 23 (patched) <https://reviews.apache.org/r/72511/#comment309511> I think if the "parse" method would return with an object which has the astTree and the tokens; then the setTokenRewrite and the other method can be done at the caller level - which would be more straightforward to me.... what do you think? - Zoltan Haindrich On May 18, 2020, 3:11 p.m., Krisztian Kasa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72511/ > ----------------------------------------------------------- > > (Updated May 18, 2020, 3:11 p.m.) > > > Review request for hive, Jesús Camacho Rodríguez and Zoltan Haindrich. > > > Bugs: HIVE-23491 > https://issues.apache.org/jira/browse/HIVE-23491 > > > Repository: hive-git > > > Description > ------- > > Move ParseDriver to parser module > > > Diffs > ----- > > parser/pom.xml 0edae27fa4 > parser/src/java/org/apache/hadoop/hive/ql/parse/ParseDriverContext.java > PRE-CREATION > parser/src/test/org/apache/hadoop/hive/ql/parse/ParseDriverContextStub.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/Context.java 318c2071d3 > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseDriver.java 46f1ec040a > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseException.java > ql/src/test/org/apache/hadoop/hive/ql/parse/TestIUD.java 4ec111cf1e > ql/src/test/org/apache/hadoop/hive/ql/parse/TestMergeStatement.java > 396d344dd6 > ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseDriver.java > ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseDriverIntervals.java > ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseWithinGroupClause.java > > > ql/src/test/org/apache/hadoop/hive/ql/parse/TestSQL11ReservedKeyWordsNegative.java > fbf2b8bb9a > > ql/src/test/org/apache/hadoop/hive/ql/parse/TestUnpermittedCharsInColumnNameCreateTableNegative.java > 02d46dc3cd > > ql/src/test/org/apache/hadoop/hive/ql/parse/positive/TestTransactionStatement.java > b13aa6855a > > > Diff: https://reviews.apache.org/r/72511/diff/1/ > > > Testing > ------- > > mvn clean install -pl parser > mvn clean install -DskipTests -pl ql > > > Thanks, > > Krisztian Kasa > >
