(bookkeeper) branch branch-4.16 updated: [Branch-4.16] Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers (#4469)

[chenhang]

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

chenhang pushed a commit to branch branch-4.16
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/branch-4.16 by this push:
     new 64d0b16793 [Branch-4.16] Fix ReadOnlyLedgerHandle leak issue when 
checkAllLedgers (#4469)
64d0b16793 is described below

commit 64d0b16793871e5c454cb574a8ef4364ae7febb0
Author: Yan Zhao <horizo...@apache.org>
AuthorDate: Wed Jul 31 11:17:28 2024 +0800

    [Branch-4.16] Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers 
(#4469)
    
    * Enhance the SimpleTestCommand test, make it cover more case.
    
    * Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers.
    
    * code clean.
---
 .../org/apache/bookkeeper/replication/AuditorCheckAllLedgersTask.java    | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/replication/AuditorCheckAllLedgersTask.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/replication/AuditorCheckAllLedgersTask.java
index 580fa60538..291561df28 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/replication/AuditorCheckAllLedgersTask.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/replication/AuditorCheckAllLedgersTask.java
@@ -210,6 +210,7 @@ public class AuditorCheckAllLedgersTask extends AuditorTask 
{
                             
auditorStats.getNumFragmentsPerLedger().registerSuccessfulValue(lh.getNumFragments());
                             
auditorStats.getNumBookiesPerLedger().registerSuccessfulValue(lh.getNumBookies());
                             auditorStats.getNumLedgersChecked().inc();
+                            lh.closeAsync();
                         });
                     } else if 
(BKException.Code.NoSuchLedgerExistsOnMetadataServerException == rc) {
                         if (LOG.isDebugEnabled()) {