----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68621/ -----------------------------------------------------------
Review request for hive, Ashutosh Chauhan, Zoltan Haindrich, and Sergey Shelukhin. Bugs: HIVE-15932 https://issues.apache.org/jira/browse/HIVE-15932 Repository: hive-git Description ------- AST was removed in explain extended in HIVE-13533; that makes sense from users perspective for the common case, but it would be useful for Hive developers and advanced users to see the AST to diagnose issues. "Explain ast" command can be added to dump the AST. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java da20d2efd8 ql/src/java/org/apache/hadoop/hive/ql/hooks/ATSHook.java fa69f13965 ql/src/java/org/apache/hadoop/hive/ql/hooks/HiveProtoLoggingHook.java 8f7c53f09e ql/src/java/org/apache/hadoop/hive/ql/parse/ExplainConfiguration.java 5ca6b59e7e ql/src/java/org/apache/hadoop/hive/ql/parse/ExplainSemanticAnalyzer.java 3aefb61ff9 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 6641e0da14 ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 15d4edfc86 ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 2566b3125c ql/src/java/org/apache/hadoop/hive/ql/plan/ExplainWork.java 3e621422e2 ql/src/test/org/apache/hadoop/hive/ql/parse/TestUpdateDeleteSemanticAnalyzer.java ec77b7376c ql/src/test/queries/clientpositive/explain_ast.q PRE-CREATION ql/src/test/results/clientpositive/explain_ast.q.out PRE-CREATION Diff: https://reviews.apache.org/r/68621/diff/1/ Testing ------- Thanks, Igor Kryvenko