Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1659#discussion_r151577088
--- Diff:
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQConnectionFactory.java
---
@@ -88,20 +83,6 @@
private boolean finalizeChecks;
- @Override
- public void writeExternal(ObjectOutput out) throws IOException {
- URI uri = toURI();
-
- try {
- out.writeUTF(uri.toASCIIString());
--- End diff --
Wouldn't this will break compatibility.
Think of anyone serializing this class.
Looking at the JNDISTorage (new class) I see that this is calling now
writeObject(getProperties());
As opposed to the writeURI.
The point was to write a string to be better compatible between versions.
Can we add back the format we had here?
---
