You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by am...@apache.org on 2015/03/19 23:40:08 UTC
drill git commit: DRILL-2488: Return DEFAULT as supported encoding
for MergeJoin since it does not currently handle SV2 or SV4.
Repository: drill
Updated Branches:
refs/heads/master 17f0befeb -> 4d398edf8
DRILL-2488: Return DEFAULT as supported encoding for MergeJoin since it does not currently handle SV2 or SV4.
Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/4d398edf
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/4d398edf
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/4d398edf
Branch: refs/heads/master
Commit: 4d398edf87d2ec6fab6c552b1f5a33fb31b955bc
Parents: 17f0bef
Author: Aman Sinha <as...@maprtech.com>
Authored: Tue Mar 17 19:23:45 2015 -0700
Committer: Aman Sinha <as...@maprtech.com>
Committed: Thu Mar 19 15:13:33 2015 -0700
----------------------------------------------------------------------
.../org/apache/drill/exec/planner/physical/MergeJoinPrel.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/drill/blob/4d398edf/exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java
index fac18c4..de5c2d5 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java
@@ -108,7 +108,9 @@ public class MergeJoinPrel extends JoinPrel {
@Override
public SelectionVectorMode[] getSupportedEncodings() {
- return SelectionVectorMode.NONE_AND_TWO;
+ // currently, MergeJoin operator is not handling incoming batch containing SV2 or SV4, so
+ // it requires a SVRemover
+ return SelectionVectorMode.DEFAULT;
}
@Override