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

    https://github.com/apache/drill/pull/1045#discussion_r162246538
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/batch/BaseRawBatchBuffer.java
 ---
    @@ -184,7 +184,7 @@ public RawFragmentBatch getNext() throws IOException {
           return null;
         }
     
    -    if (context.isOverMemoryLimit()) {
    +    if (context.getAllocator().isOverLimit()) {
    --- End diff --
    
    This one is fine. We must expose an allocator even for testing.


---

Reply via email to