Revert "HIVE-14303: CommonJoinOperator.checkAndGenObject should return directly at CLOSE state to avoid NPE if ExecReducer.close is called twice. (Zhihai Xu, reviewed by Xuefu Zhang)"
This reverts commit 529814e3d0bfaefdf3f1625d014d9e05149f5ccf, due to test failures.
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java b/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
index 5446c40..43231af 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
@@ -682,12 +682,6 @@
}
protected void checkAndGenObject() throws HiveException {
- if (state == State.CLOSE) {
- LOG.warn("checkAndGenObject is called after operator " +
- id + " " + getName() + " closed");
- return;
- }
-
if (condn[0].getType() == JoinDesc.UNIQUE_JOIN) {
// Check if results need to be emitted.
