[
https://issues.apache.org/jira/browse/HADOOP-16226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16808351#comment-16808351
]
Hudson commented on HADOOP-16226:
---------------------------------
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #16330 (See
[https://builds.apache.org/job/Hadoop-trunk-Commit/16330/])
HADOOP-16226. new Path(String str) does not remove all the trailing (aajisaka:
rev aaaf856f4b7b53d424eb1eab010311de0d5fbe1e)
* (edit)
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/Path.java
* (edit)
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestPath.java
> new Path(String str) does not remove all the trailing slashes of str
> --------------------------------------------------------------------
>
> Key: HADOOP-16226
> URL: https://issues.apache.org/jira/browse/HADOOP-16226
> Project: Hadoop Common
> Issue Type: Bug
> Components: fs
> Reporter: Akira Ajisaka
> Assignee: Akira Ajisaka
> Priority: Minor
> Fix For: 3.3.0, 3.2.1
>
> Attachments: HADOOP-16226.001.patch, HADOOP-16226.002.patch
>
>
> new Path(String str) calls Path.normalizePath to normalize the str, however,
> it does not remove all the slashes of the str.
> {code:title=Path.java}
> /**
> * Normalize a path string to use non-duplicated forward slashes as
> * the path separator and remove any trailing path separators.
> *
> * @param scheme the URI scheme. Used to deduce whether we
> * should replace backslashes or not
> * @param path the scheme-specific part
> * @return the normalized path string
> */
> private static String normalizePath(String scheme, String path) {
> {code}
> Javadoc says "remove any trailing path separators", but it removes only one
> trailing slash.
> Credit: This issue was found by [~ayushtkn] in HDFS-14369.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
