This is an automated email from the ASF dual-hosted git repository. yong pushed a commit to branch branch-4.15 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
commit 39275588122d7bb2e554c1bd3c45aa7119b2c497 Author: Yong Zhang <[email protected]> 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 (cherry picked from commit bf9a5cf3ee457797542701f0f336bede227c7fbd) --- .../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 8c0cb0bf90..8d6062d029 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 @@ -116,9 +116,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)); - } }); }
