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

[sohami]

Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1227#discussion_r183181073
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/HashJoinBatch.java
 ---
    @@ -560,6 +554,40 @@ public void close() {
         super.close();
       }
     
    +  @Override
    +  protected void updateBatchMemoryManagerStats() {
    +    stats.setLongStat(Metric.LEFT_INPUT_BATCH_COUNT, 
batchMemoryManager.getNumIncomingBatches(LEFT_INDEX));
    --- End diff --
    
    Should be fine.


---