vineetgarg02 commented on a change in pull request #1472:
URL: https://github.com/apache/hive/pull/1472#discussion_r492229067
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
##########
@@ -387,6 +387,12 @@
protected volatile boolean disableJoinMerge = false;
protected final boolean defaultJoinMerge;
+ /*
+ * This is used by prepare/execute statement
+ * Prepare/Execute requires operators to be copied and cached
+ */
+ protected Map<String, TableScanOperator> topOpsCopy = null;
Review comment:
Original operator tree shape is changed when going through physical
transformations and task generation (don't know why though), as a result this
operator tree can not be used later to regenerate tasks or re-running physical
transformations. Therefore we make a copy and cache it after operator tree is
generated.
I will leave a comment.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
