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

Sean Mackrory commented on HADOOP-15860:
----------------------------------------

So the issue isn't that path is null, the issue is that path.length == 0. path 
references a valid String object, but it's trying to do .charAt(-1), which is 
invalid. I suspect that what's happening is that you're hitting root, at which 
point it's an empty string. Maybe you can just replace the original null check 
with "while (!path.isRoot())". path.getParent() is null at that point, but we 
won't even get that far if we have bounds checking problems. We also need to 
make sure you can run the ABFS tests.

> ABFS: Throw IllegalArgumentException when Directory/File name ends with a 
> period(.)
> -----------------------------------------------------------------------------------
>
>                 Key: HADOOP-15860
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15860
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/adl
>    Affects Versions: 3.2.0
>            Reporter: Sean Mackrory
>            Assignee: Shweta
>            Priority: Major
>         Attachments: HADOOP-15860.001.patch, HADOOP-15860.002.patch, 
> HADOOP-15860.003.patch, HADOOP-15860.004.patch, HADOOP-15860.005.patch, 
> trailing-periods.patch
>
>
> If you create a directory with a trailing period (e.g. '/test.') the period 
> is silently dropped, and will be listed as simply '/test'. '/test.test' 
> appears to work just fine.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to