Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2313#discussion_r217442584
--- Diff:
artemis-ra/src/main/java/org/apache/activemq/artemis/ra/inflow/ActiveMQActivationSpec.java
---
@@ -406,7 +406,9 @@ public void setAcknowledgeMode(final String value) {
} else if ("AUTO_ACKNOWLEDGE".equalsIgnoreCase(value) ||
"Auto-acknowledge".equalsIgnoreCase(value)) {
acknowledgeMode = Session.AUTO_ACKNOWLEDGE;
} else {
- throw new IllegalArgumentException("Unsupported acknowledgement
mode " + value);
+ final String message = "Unsupported acknowledgement mode " +
value;
+ logger.warn(message);
--- End diff --
As warn requires a registered logger and logger code
---
