[jira] [Commented] (DRILL-2933) RecordBatchLoader.load(...) calls catch SchemaChangeException that load(...) never actually throws
[ https://issues.apache.org/jira/browse/DRILL-2933?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17444211#comment-17444211 ] Vitalii Diravka commented on DRILL-2933: removing _throws SchemaChangeException_ according to the _new RecordBatchLoader.load(...)_ in DRILL-8046 > 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 > Components: Execution - Relational Operators >Reporter: Daniel Barclay >Priority: Major > Fix For: Future > > > 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 (v8.20.1#820001)
[jira] [Commented] (DRILL-2933) RecordBatchLoader.load(...) calls catch SchemaChangeException that load(...) never actually throws
[ 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)