[HADOOP-16226] new Path(String str) does not remove all the trailing slashes of str

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.3.0, 3.2.1
• Component/s: fs
• Target Version/s: 3.3.0, 3.2.1
• Hadoop Flags: Reviewed
• Assignee: Akira Ajisaka
• Reporter: Akira Ajisaka
• Created: 01/Apr/19 23:59
• Updated: 05/Apr/20 08:54
• Resolved: 03/Apr/19 04:19

Description

new Path(String str) calls Path.normalizePath to normalize the str, however, it does not remove all the slashes of the str.

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

Javadoc says "remove any trailing path separators", but it removes only one trailing slash.

Credit: This issue was found by Ayush Saxena in HDFS-14369.

Issue Links

• is depended upon by: HDFS-14369 RBF: Fix trailing "/" for webhdfs
• relates to: HADOOP-16957 NodeBase.normalize doesn't removing all trailing slashes.