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

Chuan Liu commented on HADOOP-9887:
-----------------------------------

bq. do you think it would make more sense to fix this in 
Globber#getPathComponents, conditional on whether or not the pattern was 
absolute like in the old FileSystem#globStatusInternal?

One problem is that we will lost the drive letter from the components; and we 
will list directories from "/" instead of "/C:/". This has two effects: first, 
we could list from a wrong drive if the path passed in is not on the current 
drive; second, the result will be paths of the pattern without the drive 
letter. So I would still prefer the current approach. I can modify the comments 
to make it better fit the {{Globber#getPathComponents}} method. What do you 
think?
                
> globStatus does not correctly handle paths starting with a drive spec on 
> Windows
> --------------------------------------------------------------------------------
>
>                 Key: HADOOP-9887
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9887
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 3.0.0, 2.3.0
>            Reporter: Chris Nauroth
>            Assignee: Chuan Liu
>         Attachments: HADOOP-9887-trunk.patch
>
>
> Recent file system changes have caused globStatus to stop working for paths 
> starting with a drive spec on Windows.  The problem is most easily visible by 
> running {{TestFileUtil#createJarWithClassPath}} on Windows.  This method 
> attempts a globStatus with pattern {{*\{.jar,.JAR\}}}, and it no longer 
> correctly identifies files at the path ending in .jar or .JAR.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to