[GitHub] drill pull request #1227: Drill 6236: batch sizing for hash join

[Ben-Zvi]

Github user Ben-Zvi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1227#discussion_r182910955
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/HashJoinBatch.java
 ---
    @@ -346,6 +369,7 @@ public void executeBuildPhase() throws 
SchemaChangeException, ClassTransformatio
             }
             // Fall through
           case OK:
    +        batchMemoryManager.update(LEFT_INDEX);
    --- End diff --
    
    Should it be the RIGHT_INDEX here ?



---