-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34374/
-----------------------------------------------------------

Review request for drill and Steven Phillips.


Bugs: DRILL-3133
    https://issues.apache.org/jira/browse/DRILL-3133


Repository: drill-git


Description
-------

MergingRecordBatch stores batches in an array list before loading them with 
RecordBatchLoader. If the query is canceled before all received batches are 
loaded, some of the batches won't be cleaned up.

lines 307 and 339 contain questions to the reviewers. I will update the patch 
accordingly


Diffs
-----

  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/mergereceiver/MergingRecordBatch.java
 baf9bda 

Diff: https://reviews.apache.org/r/34374/diff/


Testing
-------

unit test and tpch100 are passing


Thanks,

abdelhakim deneche

Reply via email to