HBASE-22649: Encode StoreFile path URLs in the UI to handle scenarios where CF contains special characters (like # etc.)

Signed-off-by: Sakthi<sakthi@apache.org>
(cherry picked from commit dd761b1)

Author: sreenivasulureddy

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java

@@ -20,7 +20,9 @@
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.UnsupportedEncodingException;
 import java.net.InetSocketAddress;
+import java.net.URLEncoder;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
 import java.util.Collection;
@@ -282,6 +284,17 @@ public Path getPath() {
     return this.fileInfo.getPath();
   }
 
+  /**
+   * @return Encoded Path if this StoreFile was made with a Stream.
+   */
+  public Path getEncodedPath() {
+    try {
+      return new Path(URLEncoder.encode(fileInfo.getPath().toString(), HConstants.UTF8_ENCODING));
+    } catch (UnsupportedEncodingException ex) {
+      throw new RuntimeException("URLEncoder doesn't support UTF-8", ex);
+    }
+  }
+
   /**
    * @return Returns the qualified path of this StoreFile
    */
@@ -377,7 +390,7 @@ public static long getMaxSequenceIdInList(Collection<StoreFile> sfs) {
   /**
    * Check if this storefile was created by bulk load.
    * When a hfile is bulk loaded into HBase, we append
-   * '_SeqId_<id-when-loaded>' to the hfile name, unless
+   * '_SeqId_&lt;id-when-loaded&gt;' to the hfile name, unless
    * "hbase.mapreduce.bulkload.assign.sequenceNumbers" is
    * explicitly turned off.
    * If "hbase.mapreduce.bulkload.assign.sequenceNumbers"

hbase-server/src/main/resources/hbase-webapps/regionserver/region.jsp

@@ -112,7 +112,7 @@
          </tr>
        <%   for(StoreFile sf : storeFiles) { %>
          <tr>
-           <td><a href="storeFile.jsp?name=<%= sf.getPath() %>"><%= sf.getPath() %></a></td>
+           <td><a href="storeFile.jsp?name=<%= sf.getEncodedPath() %>"><%= sf.getPath() %></a></td>
            <td><%= (int) (rs.getFileSystem().getLength(sf.getPath()) / 1024 / 1024) %></td>
            <td><%= new Date(sf.getModificationTimeStamp()) %></td>
          </tr>