[
https://issues.apache.org/jira/browse/HBASE-29970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18069414#comment-18069414
]
Hudson commented on HBASE-29970:
--------------------------------
Results for branch master
[build #30 on
builds.a.o|https://ci-hbase.apache.org/job/HBase-Integration-Test/job/master/30/]:
(/) *{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}
> SplitSuccess and SplitTime metrics are no longer used at RegionServer and
> Table level
> -------------------------------------------------------------------------------------
>
> Key: HBASE-29970
> URL: https://issues.apache.org/jira/browse/HBASE-29970
> Project: HBase
> Issue Type: Improvement
> Components: metrics
> Reporter: Xiao Liu
> Assignee: Xiao Liu
> Priority: Major
> Labels: pull-request-available
> Fix For: 2.7.0, 3.0.0-beta-2, 2.5.15, 2.6.6
>
>
> This change removes the unused {{{}incrSplitSuccess(){}}},
> {{{}updateSplitTime(){}}}, and related split success/time metrics from
> RegionServer and Table level metrics sources.
> These metrics {{splitSuccess}} and {{splitTime}} is introduced by
> HBASE-12779, the split operation has been migrated from RegionServer-driven
> local execution to Master-driven Procedure-based coordination. When a
> RegionServer triggers a split due to region size threshold, it only sends a
> {{READY_TO_SPLIT}} state transition request to the Master via
> {{{}reportRegionStateTransition(){}}}. The Master then creates a
> {{SplitTableRegionProcedure}} to asynchronously coordinate the entire split
> process. The RegionServer has no knowledge of when (or whether) the split
> completes successfully, so it cannot update split success or timing metrics,
> in HBASE-16549 we refactor split metric logic and start to use
> {{splitProcMetrics}} tracking the split behaivor
> h3. Current Behavior
> * Split request metrics ({{{}splitRequestCount{}}}) are properly updated
> when a split is initiated (RS level)
> * Split success/time metrics at RegionServer and Table level were defined
> but never called
> * Master-side split metrics ({{{}splitSubmittedCount{}}}, {{{}splitTime{}}},
> {{{}splitFailedCount{}}}) are correctly maintained via the Procedure
> framework's {{getProcedureMetrics()}} mechanism
> h3. Changes Made
> * Removed {{incrSplitSuccess()}} and {{updateSplitTime()}} methods from
> {{MetricsRegionServerSource}} and {{{}MetricsRegionServerSourceImpl{}}},
> corresponding methods from {{MetricsTableSource}} and
> {{{}MetricsTableSourceImpl{}}}, and related metric definitions and histogram
> variables
> * We keep {{SplitRequest}} and support table-level updates for this metic to
> track split caused by region size exceeds the configured max size
>
> Call chain
> {code:java}
> CompactSplit.requestSplit()
> → new SplitRequest(r, midKey, server, user)
> → splits.execute(splitRequest)
> → SplitRequest.run()
> → doSplitting()
> → server.getMetrics().incrSplitRequest(tableName)
> {code}
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
