Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2035#discussion_r183194331
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/ActiveMQServerControlImpl.java
 ---
    @@ -1873,7 +1886,9 @@ public String listAllSessionsAsJSON() throws 
Exception {
           try {
              Set<ServerSession> sessions = server.getSessions();
              for (ServerSession sess : sessions) {
    -            JsonObjectBuilder obj = 
JsonLoader.createObjectBuilder().add("sessionID", 
sess.getName()).add("creationTime", 
sess.getCreationTime()).add("consumerCount", sess.getServerConsumers().size());
    +            JsonObjectBuilder obj = 
JsonLoader.createObjectBuilder().add("sessionID", sess.getName())
    +               .add("creationTime", sess.getCreationTime())
    --- End diff --
    
    Please avoid reformatting like this, there's no code change here 


---

Reply via email to