deniskuzZ commented on code in PR #5613:
URL: https://github.com/apache/hive/pull/5613#discussion_r1930773968
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java:
##########
@@ -330,6 +335,24 @@ 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() {
+ if (queryProperties == null) {
+ queryProperties = new QueryProperties();
+ }
+ queryProperties.setTablesQueried(ctx.getParsedTables().stream()
Review Comment:
use `CacheTableHelper#getUniqueNames()`
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
