Github user amansinha100 commented on a diff in the pull request:
https://github.com/apache/drill/pull/1025#discussion_r150310556
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/mergereceiver/MergingRecordBatch.java
---
@@ -177,11 +177,11 @@ public IterOutcome innerNext() {
}
boolean schemaChanged = false;
- if (prevBatchWasFull) {
+ if (!prevBatchNotFull) {
--- End diff --
The double negative makes it somewhat confusing. Perhaps rename the
variable to 'prevBatchHasSpace' .
---
