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

Zsolt Venczel commented on HDFS-13860:
--------------------------------------

Thank you very much [~shashikant] for reporting the issue and providing a patch!

I went through the code and I find the fix to be fine, it makes webhdfs 
behavior more consistent with the non webhdfs use cases.

One last inconsistency I found was that with your patch, path having "+" will 
be transformed to space and we should use %2B instead.
In my opinion this is a compromise we can live with especially if it's 
documented.

I'd +1 (non-binding) patch 01 with some documentation update.

> Space character in the path is shown as "+" while creating dirs in WebHDFS 
> ---------------------------------------------------------------------------
>
>                 Key: HDFS-13860
>                 URL: https://issues.apache.org/jira/browse/HDFS-13860
>             Project: Hadoop HDFS
>          Issue Type: Bug
>    Affects Versions: 2.10.0, 3.2.0
>            Reporter: Shashikant Banerjee
>            Assignee: Shashikant Banerjee
>            Priority: Major
>         Attachments: HDFS-13860.00.patch, HDFS-13860.01.patch
>
>
> $ ./hdfs dfs -mkdir webhdfs://127.0.0.1/tmp1/"file 1"
> 2018-08-23 15:16:08,258 WARN util.NativeCodeLoader: Unable to load 
> native-hadoop library for your platform... using builtin-java classes where 
> applicable
> $ ./hdfs dfs -ls webhdfs://127.0.0.1/tmp1
> 2018-08-23 15:16:21,244 WARN util.NativeCodeLoader: Unable to load 
> native-hadoop library for your platform... using builtin-java classes where 
> applicable
> Found 1 items
> drwxr-xr-x   - sbanerjee hadoop          0 2018-08-23 15:16 
> webhdfs://127.0.0.1/tmp1/file+1



--
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