virajjasani commented on a change in pull request #1485: HBASE-23969 Meta 
browser should show all `info` columns
URL: https://github.com/apache/hbase/pull/1485#discussion_r406753589
 
 

 ##########
 File path: 
hbase-client/src/main/java/org/apache/hadoop/hbase/MetaTableAccessor.java
 ##########
 @@ -966,6 +966,22 @@ public static ServerName getServerName(final Result r, 
final int replicaId) {
     }
   }
 
+  /**
+   * Returns a host:port string of server where the region is transitioning on.
+   * It reads "sn" column from {@link Result}, a row in catalog table.
+   * @param r Result to pull from
+   * @return A host:port string or null if necessary fields not found or empty.
+   */
+  @Nullable
+  @InterfaceAudience.Private // for use by RegionReplicaInfo which is used by 
meta browser
+  public static String getTransitioningOnServerName(final Result r, final int 
replicaId) {
 
 Review comment:
   A small test case to cover this would be good. Maybe in `TestMaster`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to