[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

[paul-rogers]

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

    https://github.com/apache/drill/pull/1023#discussion_r150345195
  
    --- Diff: pom.xml ---
    @@ -442,7 +442,7 @@
                   
-Dorg.apache.drill.exec.server.Drillbit.system_options="org.apache.drill.exec.compile.ClassTransformer.scalar_replacement=on"
                   -Ddrill.test.query.printing.silent=true
                   -Ddrill.catastrophic_to_standard_out=true
    -              -XX:MaxPermSize=512M -XX:MaxDirectMemorySize=3072M
    +              -XX:MaxPermSize=512M -XX:MaxDirectMemorySize=4096M
    --- End diff --
    
    I'm in favor of 1 for now, then we can revert it, if needed, once the batch 
size work is available and we remove the offending test (option 3).


---