rsOperationDetails UI fixes

Author: Ray Mattingly

hbase-client/src/main/java/org/apache/hadoop/hbase/client/OnlineLogRecord.java

@@ -378,7 +378,7 @@ public String toString() {
       .append("connectionAttributes", connectionAttributes).toString();
   }
 
-  public static Map<String, String> deserializeAttributes(Map<String, byte[]> attributes) {
+  private static Map<String, String> deserializeAttributes(Map<String, byte[]> attributes) {
     return attributes.entrySet().stream().collect(
       Collectors.toMap(Map.Entry::getKey, entry -> Bytes.toStringBinary(entry.getValue())));
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java

@@ -2196,6 +2196,15 @@ public static SlowLogParams getSlowLogParams(Message message, boolean slowLogSca
     return new SlowLogParams(params);
   }
 
+  public static Map<String, String> deserializeAttributes(List<NameBytesPair> attributes) {
+    return deserializeAttributes(convertNameBytesPairsToMap(attributes));
+  }
+
+  public static Map<String, String> deserializeAttributes(Map<String, byte[]> attributes) {
+    return attributes.entrySet().stream().collect(
+      Collectors.toMap(Map.Entry::getKey, entry -> Bytes.toStringBinary(entry.getValue())));
+  }
+
   /**
    * Print out some subset of a MutationProto rather than all of it and its data
    * @param proto Protobuf to print out

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

@@ -25,8 +25,8 @@
   import="org.apache.hadoop.util.StringUtils"
   import="org.apache.hadoop.hbase.regionserver.HRegionServer"
   import="org.apache.hadoop.hbase.HConstants"
-  import="org.apache.hadoop.hbase.client.OnlineLogRecord.deserializeAttributes"
   import="org.apache.hadoop.hbase.shaded.protobuf.generated.TooSlowLog"
+  import="org.apache.hadoop.hbase.shaded.protobuf.ProtobufUtil"
   import="org.apache.hadoop.hbase.namequeues.NamedQueueRecorder"
   import="org.apache.hadoop.hbase.namequeues.RpcLogDetails"
   import="org.apache.hadoop.hbase.namequeues.request.NamedQueueGetRequest"
@@ -130,8 +130,8 @@
              <td><%=r.getMultiServiceCalls()%></td>
              <td><%=r.getCallDetails()%></td>
              <td><%=r.getParam()%></td>
-             <td><%=deserializeAttributes(r.getRequestAttributes())%></td>
-             <td><%=deserializeAttributes(r.getConnectionAttributes())%></td>
+             <td><%=ProtobufUtil.deserializeAttributes(r.getRequestAttributeList())%></td>
+             <td><%=ProtobufUtil.deserializeAttributes(r.getConnectionAttributeList())%></td>
             </tr>
             <% } %>
           <% } %>
@@ -177,8 +177,8 @@
              <td><%=r.getMultiServiceCalls()%></td>
              <td><%=r.getCallDetails()%></td>
              <td><%=r.getParam()%></td>
-             <td><%=deserializeAttributes(r.getRequestAttributes())%></td>
-             <td><%=deserializeAttributes(r.getConnectionAttributes())%></td>
+             <td><%=ProtobufUtil.deserializeAttributes(r.getRequestAttributeList())%></td>
+             <td><%=ProtobufUtil.deserializeAttributes(r.getConnectionAttributeList())%></td>
             </tr>
             <% } %>
           <% } %>