[
https://issues.apache.org/jira/browse/HBASE-30505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18137635#comment-18137635
]
Hudson commented on HBASE-30505:
--------------------------------
Results for branch branch-3.0
[build #43 on
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-3.0/43/]:
(x) *{color:red}-1 overall{color}*
----
details (if available):
(/) {color:green}+1 general checks{color}
-- For more information [see general
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-3.0/43/General_20Nightly_20Build_20Report/]
(x) {color:red}-1 jdk17 hadoop3 checks{color}
-- For more information [see jdk17
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-3.0/43/JDK17_20Nightly_20Build_20Report_20_28Hadoop3_29/]
(x) {color:red}-1 jdk21 hadoop3 checks{color}
-- For more information [see jdk21
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-3.0/43/JDK21_20Nightly_20Build_20Report_20_28Hadoop3_29/]
> Incorrect BYTES_IN_REMOTE_RESULTS scan metric in async scan
> -----------------------------------------------------------
>
> Key: HBASE-30505
> URL: https://issues.apache.org/jira/browse/HBASE-30505
> Project: HBase
> Issue Type: Bug
> Components: Client
> Reporter: Xiao Liu
> Assignee: Xiao Liu
> Priority: Minor
> Labels: pull-request-available
> Fix For: 4.0.0-alpha-1, 2.7.0, 3.1.0, 3.0.1, 2.6.8, 2.5.17
>
>
> {\{AsyncScanSingleRegionRpcRetryingCaller#onComplete}} passes
> \{{isHeartbeatMessage}} as the \{{isRegionServerRemote}} argument of
> \{{ConnectionUtils#updateResultsMetrics}}:
> {code:java}
> updateResultsMetrics(scanMetrics, rawResults, isHeartbeatMessage);
> {code}
> So whether results are counted in \{{BYTES_IN_REMOTE_RESULTS}} depends on
> whether the response is a heartbeat, not on where the region server is:
> * Results from a remote region server in a normal response are not counted.
> The metric stays at or near 0.
> * Results in a heartbeat response from a local region server are counted as
> remote.
> {\{BYTES_IN_RESULTS}} is not affected.
> The bug was introduced by HBASE-17691. \{{ScannerCallable}}, changed in the
> same commit, passes the correct flag. On branch-2, \{{Table}} scans still use
> \{{ScannerCallable}}, so only \{{AsyncTable}} scans are affected. On branch-3
> and master, \{{Table}} is implemented on top of \{{AsyncTable}}, so all scans
> are affected. The wrong value shows up in \{{ResultScanner#getScanMetrics}}
> and in the MapReduce job counters.
> The fix is to pass \{{regionServerRemote}}, which the RPC call and retry
> metrics in the same class already use. HBASE-30495 fixed a similar issue for
> \{{REMOTE_RPC_RETRIES}}.
--
This message was sent by Atlassian Jira
(v10.3.26#10030026)
