[jira] [Commented] (DRILL-2618) BasicFormatMatcher calls getFirstPath(...) without checking # of paths is not zero

["Vitalii Diravka (JIRA)" <ji...@apache.org>]

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

Vitalii Diravka commented on DRILL-2618:
----------------------------------------

The functionality of this was changed in scope of DRILL-5546 and -DRILL-4185.
{code}
0: jdbc:drill:zk=local> use dfs.tmp;
+-------+--------------------------------------+
|  ok   |               summary                |
+-------+--------------------------------------+
| true  | Default schema changed to [dfs.tmp]  |
+-------+--------------------------------------+
1 row selected (0.099 seconds)
0: jdbc:drill:zk=local> select * from `emptyfolder`;
+--+
|  |
+--+
+--+
No rows selected (1.75 seconds)
{code}


Starting form Drill 1.13 the empty directory is a schemaless table in Drill, see more:
https://drill.apache.org/docs/data-sources-and-file-formats-introduction/#schemaless-tables

> BasicFormatMatcher calls getFirstPath(...) without checking # of paths is not zero
> ----------------------------------------------------------------------------------
>
>                 Key: DRILL-2618
>                 URL: https://issues.apache.org/jira/browse/DRILL-2618
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - Other
>            Reporter: Daniel Barclay
>            Assignee: Hanifi Gunes
>            Priority: Major
>             Fix For: 1.4.0
>
>
> {{BasicFormatMatcher.isReadable(...)}} calls {{getFirstPath(...)}} without checking that there is at least one path.  This can cause an IndexOutOfBoundsException.
> To reproduce, create an empty directory {{/tmp/CaseInsensitiveColumnNames}} and run {{exec/java-exec/src/test/java/org/apache/drill/TestExampleQueries.java}}.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)