[HBASE-28354] RegionSizeCalculator throws NPE when regions are in transition

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.6.0, 4.0.0-alpha-1, 3.0.0-beta-2, 2.5.9
• Labels: pull-request-available
• Assignee: Ahmad Alhour
• Reporter: Bryan Beaudreault
• Created: 09/Feb/24 13:44
• Updated: 01/Mar/24 20:04
• Resolved: 01/Mar/24 05:26

Description

When a region is in transition, it may briefly have a null ServerName in meta. The RegionSizeCalculator calls RegionLocator.getAllRegionLocations() and does not handle the possibility that a RegionLocation.getServerName() could be null. The ServerName is eventually passed into an Admin call, which results in an NPE.

This has come up in other contexts. For example, taking a look at getAllRegionLocations() impl, we have checks to ensure that we don't call null server names. We need to similarly handle the possibility of nulls in RegionSizeCalculator.

Issue Links

• relates to: HBASE-28399 region size can be wrong from RegionSizeCalculator
• links to: GitHub Pull Request #5699
• links to: GitHub Pull Request #5727
• links to: GitHub Pull Request #5728
• links to: GitHub Pull Request #5730
• links to: GitHub Pull Request #5731