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.---
