[incubator-uniffle] branch master updated: [BUG] Fix incorrect block info statistics after read finished (#401)

[roryqi]

This is an automated email from the ASF dual-hosted git repository.

roryqi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 9b10545b [BUG] Fix incorrect block info statistics after read finished 
(#401)
9b10545b is described below

commit 9b10545b53f3c39818c6431ebb408421ee48204f
Author: xianjingfeng <583872...@qq.com>
AuthorDate: Mon Dec 12 19:57:28 2022 +0800

    [BUG] Fix incorrect block info statistics after read finished (#401)
    
    ### What changes were proposed in this pull request?
    Fix incorrect block info statistics after read finished
    
    ### Why are the changes needed?
    It is bug introduce by #395
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    No need
---
 .../uniffle/storage/handler/impl/MultiReplicaClientReadHandler.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/storage/src/main/java/org/apache/uniffle/storage/handler/impl/MultiReplicaClientReadHandler.java
 
b/storage/src/main/java/org/apache/uniffle/storage/handler/impl/MultiReplicaClientReadHandler.java
index c2e766a9..e0408af9 100644
--- 
a/storage/src/main/java/org/apache/uniffle/storage/handler/impl/MultiReplicaClientReadHandler.java
+++ 
b/storage/src/main/java/org/apache/uniffle/storage/handler/impl/MultiReplicaClientReadHandler.java
@@ -85,7 +85,7 @@ public class MultiReplicaClientReadHandler extends 
AbstractClientReadHandler {
   @Override
   public void updateConsumedBlockInfo(BufferSegment bs, boolean 
isSkippedMetrics) {
     super.updateConsumedBlockInfo(bs, isSkippedMetrics);
-    handlers.get(Math.max(readHandlerIndex, handlers.size() - 
1)).updateConsumedBlockInfo(bs, isSkippedMetrics);
+    handlers.get(Math.min(readHandlerIndex, handlers.size() - 
1)).updateConsumedBlockInfo(bs, isSkippedMetrics);
   }
 
   @Override