jeanouii commented on code in PR #1736:
URL: https://github.com/apache/activemq/pull/1736#discussion_r2919782690
##########
activemq-client/src/main/java/org/apache/activemq/ActiveMQConnection.java:
##########
@@ -150,7 +150,7 @@ public class ActiveMQConnection implements Connection,
TopicConnection, QueueCon
private boolean optimizeAcknowledge;
private long optimizeAcknowledgeTimeOut = 0;
private long optimizedAckScheduledAckInterval = 0;
- private boolean nestedMapAndListEnabled = true;
+ private boolean nestedMapAndListEnabled = false;
Review Comment:
I like the idea of not having a specific flag just for this. We are just
starting working on JMS 3.1 compliance and we may encounter other situations so
it may well end up by having many flags.
Having one flag for turning the entire broker into a compatible mode seems
relevant to me.
We could as well force (and log of course) `nestedMapAndListEnabled` to
`false` when the new strictCompliance flag is enabled? Thoughts?
--
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]
For further information, visit: https://activemq.apache.org/contact
