marton-bod commented on a change in pull request #2261:
URL: https://github.com/apache/hive/pull/2261#discussion_r636151988
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/QueryState.java
##########
@@ -59,6 +60,11 @@
static public final String USERID_TAG = "userid";
+ /**
+ * map of tables involved in the query.
+ */
+ private final Map<String, Object> tableMap = new HashMap<>();
Review comment:
Can we make this a generic map so we can store things other than tables
here? We will need this container also for other info that - in the absence of
this new query state feature - we previously had to force into the conf, such
as write commit info (jobID, vertexID, taskNum), CTAS info (is the query ctas?
what's the ctas target table name), etc.
--
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.
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
