Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1181#discussion_r178445244
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java
 ---
    @@ -127,17 +153,20 @@
         @Override
         public void update(int inputIndex) {
           switch(inputIndex) {
    -        case 0:
    -          leftSizer = new RecordBatchSizer(left);
    -          leftRowWidth = leftSizer.netRowWidth();
    +        case LEFT_INDEX:
    +          setRecordBatchSizer(inputIndex, new RecordBatchSizer(left));
    +          leftRowWidth = getRecordBatchSizer(inputIndex).netRowWidth();
    +          logger.debug("mergejoin left incoming batch sizer : {}", 
getRecordBatchSizer(inputIndex));
    --- End diff --
    
    Maybe "Left incoming batch size"? (The log will tell you it is the merge 
join. The "sizer" is an implementation detail.)


---

Reply via email to