(bookkeeper) branch master updated: Remove the unused logs in the CleanupLedgerManager.recordPromise (#4121)

[shoothzj]

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

shoothzj 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 bf9a5cf3ee Remove the unused logs in the 
CleanupLedgerManager.recordPromise (#4121)
bf9a5cf3ee is described below

commit bf9a5cf3ee457797542701f0f336bede227c7fbd
Author: Yong Zhang <zhangyong1025...@gmail.com>
AuthorDate: Wed Nov 1 22:36:26 2023 +0800

    Remove the unused logs in the CleanupLedgerManager.recordPromise (#4121)
    
    ---
    
    ### Motivation
    
    In the cleanup ledger manager, we just record every future and remove
    them when the future complete. Seems we don't need to print the logs
---
 .../src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java
index f9c3a00d68..669304c1c1 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java
@@ -115,9 +115,6 @@ public class CleanupLedgerManager implements LedgerManager {
         futures.add(promise);
         promise.whenComplete((result, exception) -> {
             futures.remove(promise);
-            if (exception != null) {
-                log.error("Failed on operating ledger metadata: {}", 
BKException.getExceptionCode(exception));
-            }
         });
     }