[GitHub] [drill] vdiravka commented on pull request #1455: DRILL-6724: Dump operator context to logs when error occurs during query execution

[GitHub]

I still think that it is better to use existing BatchState or IterOutcome or 
even maybe exceptionState in `IteratorValidatorBatchIterator`. Why do we need 
two states for batch? In what cases they can be different?
Looks like it is possible to set IterOutcome to STOP in case of any exception 
and then batches will be dumped.

[ Full content available at: https://github.com/apache/drill/pull/1455 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org