eolivelli commented on code in PR #17736:
URL: https://github.com/apache/pulsar/pull/17736#discussion_r977250234


##########
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java:
##########
@@ -3639,6 +3664,7 @@ private void checkManagedLedgerIsOpen() throws 
ManagedLedgerException {
     }
 
     synchronized void setFenced() {
+        log.info("{} Moving to Fenced state", name);

Review Comment:
   This is not a "problem".
   it may happen, and we are handling it safely.
   there is nothing that the sysadmin should be afraid of.
   
   we should log "WARN" or "ERROR" when there is something bad, and you have to 
take extra care



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to