You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Bryan Beaudreault (Jira)" <ji...@apache.org> on 2022/03/02 15:54:00 UTC

[jira] [Created] (HBASE-26790) getAllRegionLocations can cache locations with null hostname

Bryan Beaudreault created HBASE-26790:
-----------------------------------------

             Summary: getAllRegionLocations can cache locations with null hostname
                 Key: HBASE-26790
                 URL: https://issues.apache.org/jira/browse/HBASE-26790
             Project: HBase
          Issue Type: Bug
            Reporter: Bryan Beaudreault


RegionLocator methods typically delegate to ConnectionImplementation.locateRegion, which throws a NoServerForRegionException if the located region's serverName is null. RegionLocator.getAllRegionLocations does not go through that path, instead caching all returned region locations without any validation. This can result in a "dirty" meta cache, since clients do not expect to have null serverNames in the meta cache. We should add the same throwing of NoServerForRegionException to this method as used in the others. Or at least we should not cache the result if the serverName is null.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)