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

ayegorov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new b3ce6ca  fix a metric error in bookieStats
b3ce6ca is described below

commit b3ce6ca8950da29d1052c9cb230e8e51df9c3d10
Author: ken <[email protected]>
AuthorDate: Sat Mar 12 04:13:16 2022 +0800

    fix a metric error in bookieStats
    
    Descriptions of the changes in this PR:
    
    
    
    ### Motivation
    
    fix a metric error in bookieStats
    
    ### Changes
    
    getReadEntryStats().registerFailedValue(entrySize) -> 
getReadBytesStats().registerFailedValue(entrySize)
    
    
    
    
    
    
    Reviewers: Andrey Yegorov <None>, Nicolò Boschi <[email protected]>
    
    This closes #3083 from TakaHiro0208/fix_bookieStats_metric_error
---
 .../src/main/java/org/apache/bookkeeper/bookie/BookieImpl.java          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieImpl.java 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieImpl.java
index b382c4d..9bc084e 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieImpl.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieImpl.java
@@ -1095,7 +1095,7 @@ public class BookieImpl extends BookieCriticalThread 
implements Bookie {
                 
bookieStats.getReadBytesStats().registerSuccessfulValue(entrySize);
             } else {
                 
bookieStats.getReadEntryStats().registerFailedEvent(elapsedNanos, 
TimeUnit.NANOSECONDS);
-                bookieStats.getReadEntryStats().registerFailedValue(entrySize);
+                bookieStats.getReadBytesStats().registerFailedValue(entrySize);
             }
         }
     }

Reply via email to