ddanielr commented on code in PR #3177:
URL: https://github.com/apache/accumulo/pull/3177#discussion_r1095054304
##########
server/base/src/main/java/org/apache/accumulo/server/util/SystemPropUtil.java:
##########
@@ -74,7 +76,13 @@ private static String validateSystemProperty(String
property, final String value
if (!Property.isValidZooPropertyKey(property)) {
IllegalArgumentException iae =
new IllegalArgumentException("Zookeeper property is not mutable: " +
property);
- log.debug("Attempted to set zookeeper property. It is not mutable",
iae);
+ log.error("Encountered error setting zookeeper property", iae);
Review Comment:
@EdColeman confirmed that the calling method does in fact log the exceptions
at the `error` level.
I've opened https://github.com/apache/accumulo/pull/3184 to revert this
change.
--
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]
