[jira] [Commented] (DRILL-2933) RecordBatchLoader.load(...) calls catch SchemaChangeException that load(...) never actually throws

["Daniel Barclay (Drill) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DRILL-2933?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14540512#comment-14540512 ] 

Daniel Barclay (Drill) commented on DRILL-2933:
-----------------------------------------------

Note:  If it is intended that RecordBatchLoader.load(...) declares that it throws SchemaChangeException in order to "reserve the right" to throw that exception in the future (e.g., to help make sure that current callers are already prepared for that case), then there's no problem (except that SchemaChangeException and load's throwing of SchemaChangeException could use at least minimal documentation).


> RecordBatchLoader.load(...) calls catch SchemaChangeException that load(...) never actually throws
> --------------------------------------------------------------------------------------------------
>
>                 Key: DRILL-2933
>                 URL: https://issues.apache.org/jira/browse/DRILL-2933
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Daniel Barclay (Drill)
>             Fix For: 1.2.0
>
>
> There are about 9 calls to RecordBatchLoader.load(...) that catch SchemaChangeException because it is declared to be thrown by RecordBatchLoader.load(...).
> However, RecordBatchLoader.load(...) never actually throws SchemaChangeException.
> (To find those calls, comment out the "throws SchemaChangeException" on RecordBatchLoader.load(...) and follow the compilation errors.)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)