saintstack opened a new pull request #1305: HBASE-23984 [Flakey Tests] TestMasterAbortAndRSGotKilled fails in tea… URL: https://github.com/apache/hbase/pull/1305 …rdown Remove Region from RegionServer#regionsInTransitionInRS Map even on exception (because server is aborted). hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Only dump metrics once on abort. Refactor closeRegion method removing overlap with CloseRegionHandler; i.e. call to CP. hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/AssignRegionHandler.java Refactor so we remove Region from regionsInTransitionRS on success or failure. hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java Cleanup but also made it so UnassignRegionHandler can use it closing Regions. hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/UnassignRegionHandler.java Use CloseRegionHandler to close Region rather than dupe code. hbase-server/src/main/java/org/apache/hadoop/hbase/util/JVMClusterUtil.java Bit more info when thread dumping... TestRegionServerNoMaster was changed because it presumes a behavior not present anymore (that CRH will leave danglng mention in regionsInTransitionInRS as it used to)
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services