saintstack commented on a change in pull request #923: HBASE-23554 Encoded 
regionname to regionname utility
URL: https://github.com/apache/hbase/pull/923#discussion_r356414281
 
 

 ##########
 File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
 ##########
 @@ -1696,24 +1695,42 @@ private void checkHFileFormatVersionForMob() throws 
IOException {
     }
   }
 
+  // This method implements Admin getRegionInfo. On RegionServer, it is
 
 Review comment:
   BIt odd. This implements and Interface that has two different 
implementations... one in RS and other in M. Seemed best as comment. Made it 
javadoc as you requested.

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