Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2254#discussion_r211417758
--- Diff:
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQMessage.java
---
@@ -371,7 +369,7 @@ public Destination getJMSReplyTo() throws JMSException {
String name = address.toString();
// swap the old prefixes for the new ones so the proper
destination type gets created
- if (enable1xPrefixes) {
+ if (session.isEnable1xPrefixes()) {
--- End diff --
Hot path here
---