ndimiduk commented on a change in pull request #1305: HBASE-23984 [Flakey Tests] TestMasterAbortAndRSGotKilled fails in tea… URL: https://github.com/apache/hbase/pull/1305#discussion_r394652253
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java ########## @@ -247,9 +249,8 @@ public static boolean TEST_SKIP_REPORTING_TRANSITION = false; /** - * A map from RegionName to current action in progress. Boolean value indicates: - * true - if open region action in progress - * false - if close region action in progress + * A map from encoded region name as bytes to current action in progress. Boolean value indicates: + * TRUE if open region action in progress, and FALSE if close region action in progress Review comment: I wish this was a map to an enum, or something like `RegionState.State`, instead of just this boolean flag. ---------------------------------------------------------------- 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
