[
https://issues.apache.org/jira/browse/HBASE-30217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18089389#comment-18089389
]
Hudson commented on HBASE-30217:
--------------------------------
Results for branch master
[build #99 on
builds.a.o|https://ci-hbase.apache.org/job/HBase-Integration-Test/job/master/99/]:
(/) *{color:green}+1 overall{color}*
----
details (if available):
(/) {color:green}+1 client integration test for 3.3.5 {color}
(/) {color:green}+1 client integration test for 3.3.5 with shaded hadoop
client{color}
(/) {color:green}+1 client integration test for 3.3.6 {color}
(/) {color:green}+1 client integration test for 3.3.6 with shaded hadoop
client{color}
(/) {color:green}+1 client integration test for 3.4.0 {color}
(/) {color:green}+1 client integration test for 3.4.0 with shaded hadoop
client{color}
(/) {color:green}+1 client integration test for 3.4.1 {color}
(/) {color:green}+1 client integration test for 3.4.1 with shaded hadoop
client{color}
(/) {color:green}+1 client integration test for 3.4.2 {color}
(/) {color:green}+1 client integration test for 3.4.2 with shaded hadoop
client{color}
(/) {color:green}+1 client integration test for 3.4.3 {color}
(/) {color:green}+1 client integration test for 3.4.3 with shaded hadoop
client{color}
> 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
> Priority: Major
> Labels: pull-request-available
> 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:
> * 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)
