jbertram commented on code in PR #5477:
URL: https://github.com/apache/activemq-artemis/pull/5477#discussion_r1934597872
##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:
##########
@@ -94,16 +94,11 @@ private enum STATE {
FluentPropertyBeanIntrospectorWithIgnores.addIgnore(ServerLocatorImpl.class.getName(),
"setThreadPools");
}
- private static final long serialVersionUID = -1615857864410205260L;
-
// This is the default value
private ClientProtocolManagerFactory protocolManagerFactory = new
ActiveMQClientProtocolManagerFactory().setLocator(this);
private final boolean ha;
- // this is not used... I'm only keeping it here because of Serialization
compatibility and Wildfly usage on JNDI.
- private boolean finalizeCheck;
Review Comment:
I just left this one alone.
--
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: gitbox-unsubscr...@activemq.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact
