drill-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From amansi...@apache.org
Subject drill git commit: DRILL-2488: Return DEFAULT as supported encoding for MergeJoin since it does not currently handle SV2 or SV4.
Date Thu, 19 Mar 2015 22:40:08 GMT
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 <asinha@maprtech.com>
Authored: Tue Mar 17 19:23:45 2015 -0700
Committer: Aman Sinha <asinha@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


Mime
View raw message