deniskuzZ commented on code in PR #5613:
URL: https://github.com/apache/hive/pull/5613#discussion_r1941930906
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java:
##########
@@ -330,6 +332,17 @@ public void analyze(ASTNode ast, Context ctx) throws
SemanticException {
initCtx(ctx);
init(true);
analyzeInternal(ast);
+ postAnalyze();
+ }
+
+ /**
+ * This method is called after analyzeInternal, assuming all the semantic
context is present (analyzeInternal is
+ * overridden by the subclasses of BaseSemanticAnalyzer).
+ * The reason why postAnalyze is private is that it's not supposed to be
overridden.
+ */
+ private void postAnalyze() {
Review Comment:
why do we need `postAnalyze`? can't we move `setUsedTables` to `endAnalysis`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
