This is an automated email from the ASF dual-hosted git repository.
sijie 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 95ee608 AbstractZkLedgerManager: wrap Log.debug in an isDebugEnabled
check
95ee608 is described below
commit 95ee60872ae6a47996dc3215ef911ce375458a72
Author: Samuel Just <[email protected]>
AuthorDate: Thu Dec 21 21:54:25 2017 -0800
AbstractZkLedgerManager: wrap Log.debug in an isDebugEnabled check
Signed-off-by: Samuel Just <sjustsalesforce.com>
Author: Samuel Just <[email protected]>
Reviewers: Sijie Guo <[email protected]>
This closes #908 from athanatos/forupstream/small-fixes-1/zklm-debug
---
.../main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java
index f5d8e7f..09c7a39 100644
---
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java
+++
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java
@@ -322,7 +322,9 @@ abstract class AbstractZkLedgerManager implements
LedgerManager, Watcher {
@Override
public void registerLedgerMetadataListener(long ledgerId,
LedgerMetadataListener listener) {
if (null != listener) {
- LOG.debug("Registered ledger metadata listener {} on ledger {}.",
listener, ledgerId);
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("Registered ledger metadata listener {} on ledger
{}.", listener, ledgerId);
+ }
Set<LedgerMetadataListener> listenerSet = listeners.get(ledgerId);
if (listenerSet == null) {
Set<LedgerMetadataListener> newListenerSet = new
HashSet<LedgerMetadataListener>();
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
