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

Neal Richardson commented on ARROW-7648:
----------------------------------------

See also ARROW-6622 and https://github.com/apache/arrow/pull/5445. Back then it 
was decided that it wasn't C++'s responsibility to sanitize file paths. Happy 
to remove the R code that patch added if we reach a different decision this 
time.

> [C++] Sanitize local paths on Windows
> -------------------------------------
>
>                 Key: ARROW-7648
>                 URL: https://issues.apache.org/jira/browse/ARROW-7648
>             Project: Apache Arrow
>          Issue Type: Wish
>          Components: C++
>            Reporter: Antoine Pitrou
>            Assignee: Antoine Pitrou
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> One way or the other, we should try to sanitize local filesystem paths on 
> Windows, by converting backslashes into regular slahes.
> One place to do it is {{FileSystemFromUri}}. One complication is that 
> \-separated paths can fail parsing as a URI, but we only want to sanitize a 
> path if we detected it's a local path (by parsing the URI). Perhaps trying on 
> error would work.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to