Skip to content

Commit

Permalink
[Branch-4.16] Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers (
Browse files Browse the repository at this point in the history
#4469)

* Enhance the SimpleTestCommand test, make it cover more case.

* Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers.

* code clean.
  • Loading branch information
horizonzy authored Jul 31, 2024
1 parent e0f23cf commit 64d0b16
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,7 @@ void checkAllLedgers() throws BKException, IOException, InterruptedException {
auditorStats.getNumFragmentsPerLedger().registerSuccessfulValue(lh.getNumFragments());
auditorStats.getNumBookiesPerLedger().registerSuccessfulValue(lh.getNumBookies());
auditorStats.getNumLedgersChecked().inc();
lh.closeAsync();
});
} else if (BKException.Code.NoSuchLedgerExistsOnMetadataServerException == rc) {
if (LOG.isDebugEnabled()) {
Expand Down

0 comments on commit 64d0b16

Please sign in to comment.