You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Xiao Liu (Jira)" <ji...@apache.org> on 2026/06/10 08:51:00 UTC

[jira] [Created] (HBASE-30217) Remove dead code getRegionId and isTemporaryDirectoryName in HRegionFileSystem

Xiao Liu created HBASE-30217:
--------------------------------

             Summary: Remove dead code getRegionId and isTemporaryDirectoryName in HRegionFileSystem
                 Key: HBASE-30217
                 URL: https://issues.apache.org/jira/browse/HBASE-30217
             Project: HBase
          Issue Type: Improvement
            Reporter: Xiao Liu
            Assignee: Xiao Liu
             Fix For: 2.7.0, 3.0.0-beta-2, 2.5.16, 2.6.7


{\{HRegionFileSystem.getRegionId(Path)}} and \{{isTemporaryDirectoryName(String)}} were introduced in HBASE-28469 for \{{DataTieringManager}} to resolve region from HFile paths. After HBASE-29727 refactored \{{BlockCacheKey}} to carry region/cf/file names directly, \{{DataTieringManager}} no longer calls these methods, leaving them as dead code.

Additionally, \{{REGION_SPLITS_DIR}} (.splits) and \{{REGION_MERGES_DIR}} (.merges) constants have not been used since HBASE-26187, which changed splits/merges to write directly into the daughter/merged region directory under the table dir.

This patch:

{quote}* Removes unused \{{getRegionId(Path)}} and \{{isTemporaryDirectoryName(String)}}
* Marks \{{REGION_SPLITS_DIR}} and \{{REGION_MERGES_DIR}} as \{{@Deprecated}} (since 3.0.0, will be removed in 4.0.0)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)