[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_r182929487
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchMemoryManager.java
 ---
    @@ -147,6 +149,12 @@ public int update(int inputIndex, int outputPosition) {
         return getOutputRowCount();
       }
     
    +  public int update(int inputIndex, int outputPosition, boolean 
useAggregate) {
    +    // by default just return the outputRowCount
    +    return update(inputIndex, outputPosition, false);
    --- End diff --
    
    Is this an infinite recursive call ??



---