pradeep85841 commented on code in PR #1736:
URL: https://github.com/apache/activemq/pull/1736#discussion_r2919557516
##########
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:
Thank you for the guidance. I completely understand your point—changing the
nestedMapAndListEnabled default is a very big change and will surely break many
legacy users who have been using Maps and Lists for years. I will revert this
default back to true.
As suggested, I will introduce a specific flag, complianceMode (or
strictCompliance), to handle the Character rejection separately. This way, we
can achieve Jakarta 3.1 compliance without affecting the long-standing behavior
of Maps and Lists.
I will update the PR with these changes and the refactored tests. Please let
me know if this sounds correct to you
--
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
