[GitHub] drill pull request #1091: DRILL-6071: Limit batch size for flatten operator

[paul-rogers]

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

    https://github.com/apache/drill/pull/1091#discussion_r162225507
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractBase.java
 ---
    @@ -29,9 +28,12 @@
     
       public static long INIT_ALLOCATION = 1_000_000L;
       public static long MAX_ALLOCATION = 10_000_000_000L;
    +  // Default output batch size, 512MB
    +  public static long OUTPUT_BATCH_SIZE = 512 * 1024 * 1024L;
    --- End diff --
    
    Also, if the size is a config parameter, shouldn't the default be set in 
the drill-module.conf file, not in a Java constant?


---