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

PJ Fanning commented on DRILL-8069:
-----------------------------------

[~cgivre]  I raised this based on our email exchange

> remove use of excel sheet getLastRowNum because it is not guaranteed to work
> ----------------------------------------------------------------------------
>
>                 Key: DRILL-8069
>                 URL: https://issues.apache.org/jira/browse/DRILL-8069
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Execution - Data Types
>            Reporter: PJ Fanning
>            Priority: Major
>
> excel-streaming-reader can only get last row num from sheets that have 
> dimension data and not all sheets have this set - it is safer to iterate 
> through the row data and not try to shortcut the reading of the sheet based 
> on getLastRowNum



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to