Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2035#discussion_r183194317
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/ActiveMQServerControlImpl.java
---
@@ -1830,7 +1830,18 @@ public String listConnectionsAsJSON() throws
Exception {
Set<RemotingConnection> connections =
server.getRemotingService().getConnections();
for (RemotingConnection connection : connections) {
- JsonObjectBuilder obj =
JsonLoader.createObjectBuilder().add("connectionID",
connection.getID().toString()).add("clientAddress",
connection.getRemoteAddress()).add("creationTime",
connection.getCreationTime()).add("implementation",
connection.getClass().getSimpleName()).add("sessionCount",
server.getSessions(connection.getID().toString()).size());
+ JsonObjectBuilder obj =
JsonLoader.createObjectBuilder().add("connectionID",
connection.getID().toString())
+ .add("clientAddress", connection.getRemoteAddress())
+ .add("creationTime", connection.getCreationTime())
+ .add("implementation",
connection.getClass().getSimpleName())
--- End diff --
Please avoid reformatting like this, there's no code change here
---
