[ https://issues.apache.org/jira/browse/HBASE-25368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17252175#comment-17252175 ]
Duo Zhang commented on HBASE-25368: ----------------------------------- This breaks TestAdmin2.testCloseRegionIfInvalidRegionNameIsPassed. Since it has not been resolved and has not been committed to other branches yet, I've reverted it from master for now. Please open a new PR to land it again. Thanks. > Filter out more invalid encoded name in isEncodedRegionName(byte[] > regionName) > ------------------------------------------------------------------------------- > > Key: HBASE-25368 > URL: https://issues.apache.org/jira/browse/HBASE-25368 > Project: HBase > Issue Type: Improvement > Components: Client > Reporter: Huaxiang Sun > Assignee: Huaxiang Sun > Priority: Major > > {code:java} > public static boolean isEncodedRegionName(byte[] regionName) { > // If not parseable as region name, presume encoded. TODO: add stringency; > e.g. if hex. > return parseRegionNameOrReturnNull(regionName) == null && regionName.length > <= MD5_HEX_LENGTH; > } > Right now, if it passes in an table name, it still thinks it is a encoded > region name and will result in unnecessary registry query for meta regions. > This can be avoided if table names can be filtered out early in this > method.{code} -- This message was sent by Atlassian Jira (v8.3.4#803005)