[ 
https://issues.apache.org/jira/browse/DRILL-2933?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Barclay (Drill) updated DRILL-2933:
------------------------------------------
    Description: 
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.)



  was:
There are about 9 calls to RecordBatchLoader.load(...) that apparently try to 
detect schema changes by catching SchemaChangeException.

However, RecordBatchLoader.load(...) never actually throws 
SchemaChangeException to indicate a schema change--it returns true to indicate 
that.

Those 9 calls do not also check the return value, so they cannot detect schema 
changes.


(To find those calls, comment out the "throws SchemaChangeException" on 
RecordBatchLoader.load(...) and follow the compilation errors.)




> RecordBatchLoader.load(...) calls catch SchemaChangeException that load(...) 
> never throws
> -----------------------------------------------------------------------------------------
>
>                 Key: DRILL-2933
>                 URL: https://issues.apache.org/jira/browse/DRILL-2933
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Daniel Barclay (Drill)
>
> 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)

Reply via email to