commit 96f9c5a1b3a0b74d641a288544b6a3a0aa986574
author kennyjiang <630862708@qq.com> Mon Jul 27 14:59:51 2020 +0800
committer GitHub <noreply@github.com> Mon Jul 27 08:59:51 2020 +0200
tree 1f52d8858421ec04dddf03f57116012ff7018ae3
parent 3748b9c554c6ba0d5b5c1780d4d35db2a83e89ef

fix fillReadAheadCache stat bug

These code blocks will not be executed:
dbLedgerStorageStats.getReadAheadBatchCountStats().registerSuccessfulValue(count);
dbLedgerStorageStats.getReadAheadBatchSizeStats().registerSuccessfulValue(size);

### Motivation

fix fillReadAheadCache stat bug

### Changes

Replace return with break to exit the while loop



Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Rajan Dhabalia <rdhabalia@apache.org>

This closes #2383 from holmes07/master

bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/SingleDirectoryDbLedgerStorage.java

diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/SingleDirectoryDbLedgerStorage.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/SingleDirectoryDbLedgerStorage.java
index 60e3d64..931c3e0 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/SingleDirectoryDbLedgerStorage.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/SingleDirectoryDbLedgerStorage.java
@@ -475,7 +475,7 @@
 
                     if (currentEntryLedgerId != orginalLedgerId) {
                         // Found an entry belonging to a different ledger, stopping read-ahead
-                        return;
+                        break;
                     }
 
                     // Insert entry in read cache