You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by se...@apache.org on 2014/08/06 23:34:45 UTC
svn commit: r1616357 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
Author: sershe
Date: Wed Aug 6 21:34:45 2014
New Revision: 1616357
URL: http://svn.apache.org/r1616357
Log:
HIVE-7574 : CommonJoinOperator.checkAndGenObject calls LOG.Trace per row from probe side in a HashMap join consuming 4% of the CPU (Mostafa Mokhtar, reviewed by Sergey Shelukhin)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java?rev=1616357&r1=1616356&r2=1616357&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java Wed Aug 6 21:34:45 2014
@@ -705,13 +705,9 @@ public abstract class CommonJoinOperator
}
if (allOne) {
- LOG.info("calling genAllOneUniqueJoinObject");
genAllOneUniqueJoinObject();
- LOG.info("called genAllOneUniqueJoinObject");
} else {
- LOG.trace("calling genUniqueJoinObject");
genUniqueJoinObject(0, 0);
- LOG.trace("called genUniqueJoinObject");
}
} else {
// does any result need to be emitted
@@ -753,17 +749,11 @@ public abstract class CommonJoinOperator
}
if (!hasEmpty && !mayHasMoreThanOne) {
- LOG.trace("calling genAllOneUniqueJoinObject");
genAllOneUniqueJoinObject();
- LOG.trace("called genAllOneUniqueJoinObject");
} else if (!hasEmpty && !hasLeftSemiJoin) {
- LOG.trace("calling genUniqueJoinObject");
genUniqueJoinObject(0, 0);
- LOG.trace("called genUniqueJoinObject");
} else {
- LOG.trace("calling genObject");
genJoinObject();
- LOG.trace("called genObject");
}
}
Arrays.fill(aliasFilterTags, (byte)0xff);