Marcosrico commented on code in PR #2201:
URL: https://github.com/apache/helix/pull/2201#discussion_r964151848
##########
helix-core/src/main/java/org/apache/helix/manager/zk/ZkBaseDataAccessor.java:
##########
@@ -726,7 +726,7 @@ public boolean remove(String path, int options) {
// case, we try to delete recursively
_zkClient.delete(path);
} catch (ZkException e) {
- LOG.debug("Failed to delete {} with opts {}, err: {}. Try recursive
delete", path, options,
+ LOG.info("Failed to delete {} with opts {}, err: {}. Try recursive
delete", path, options,
Review Comment:
Hmm you're right. What we're looking for is to diminish logging clutter -
increasing the logging level would worsen that! I wanted to have the logging
level consistent with the log error in the remove function but that wouldn't
make sense to the overall objective of this fix. Thanks for the comment!
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
