ayushtkn commented on code in PR #3485:
URL: https://github.com/apache/hive/pull/3485#discussion_r934225298
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/Operator.java:
##########
@@ -675,6 +678,13 @@ public void close(boolean abort) throws HiveException {
return;
}
+ // if the operator has not been initialized, there is no need to close
+ if (!this.rootInitializeCalled) {
+ LOG.debug("No need to close operator {}", this);
+ return;
+ }
+
+
Review Comment:
why this is specific to MR and doesn't happen with Tez? And what is the
actual scenario/query where this happens, the extended test doesn't seems to be
very indicative
--
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]
