You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Weston Pace (Jira)" <ji...@apache.org> on 2022/04/28 16:31:00 UTC

[jira] [Commented] (ARROW-16392) [C++] Substrait consumer cannot handle file URIs that contain a Windows drive letter

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

Weston Pace commented on ARROW-16392:
-------------------------------------

Right now if we see {{file:///}} we drop the first 7 characters.  When we get a URI with a drive letter:

{noformat}
file:///C:/}}
{noformat}

We need to drop the first 8 characters.  The easiest thing to do would be to check if the first segment ends in a colon and assume it's a drive letter.

> [C++] Substrait consumer cannot handle file URIs that contain a Windows drive letter
> ------------------------------------------------------------------------------------
>
>                 Key: ARROW-16392
>                 URL: https://issues.apache.org/jira/browse/ARROW-16392
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: Vibhatha Lakmal Abeykoon
>            Assignee: Vibhatha Lakmal Abeykoon
>            Priority: Major
>
> Observed the following when looking into a functional test for file reading in a Substrait Plan. 
> Referring to the existing logic here: 
> https://github.com/apache/arrow/blob/e0061bbb8cd269f4e8b880d1a4ba181312cbc07f/cpp/src/arrow/engine/substrait/relation_internal.cc#L114
> This needs to be changed to acustom support for Windows.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)