@paul-rogers, source of IOBE here is when allocating value vector buffers or 
reading records from input. Currently, every Exception (except 
OutOfMemoryException and UserException) will be wrapped into UserException with 
ErrorType INTERNAL_ERROR (and will remain being such after FragmentExecutor 
handling).
Purpose of this improvement is to enhance error messages to provide more useful 
information to users, while original cause is preserved. Provided context may 
be useful for users as it may provide some ways of resolving the issue or to 
make a workaround while the issue is being investigated etc.
You're right that IOBE indicates a program error so I've made changes to wrap 
IOBE into UserException of type INTERNAL_ERROR and not DATA_READ_ERROR (as it 
was before in the PR).

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

Reply via email to