[GitHub] [activemq-artemis] wy96f commented on a change in pull request #2677: ARTEMIS-2350 Fix ClassNotFoundException while invoking listConsumers

[GitBox <gi...@apache.org>]

wy96f commented on a change in pull request #2677: ARTEMIS-2350 Fix ClassNotFoundException while invoking listConsumers
URL: https://github.com/apache/activemq-artemis/pull/2677#discussion_r287785732
 
 

 ##########
 File path: artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/JsonUtil.java
 ##########
 @@ -275,11 +274,11 @@ public static JsonArrayBuilder toJsonArrayBuilder(byte[] byteArray) {
    }
 
    public static JsonArray readJsonArray(String jsonString) {
-      return Json.createReader(new StringReader(jsonString)).readArray();
 
 Review comment:
   Hi, clebertsuconic, we dived into code today and checked our deployed production configuration. Found that we didn't use the etc/management.xml and instead added jmx configuration in artemis.profile  like "-Dcom.sun.management.jmxremote=true -Dcom.sun.management.jmxremote.port=1099". In such a way, the rmi thread pool exeuctor is using system classloader. When requests from jconsole come in, the exception occcurs.
   
   The reason ActiveMQServerControlTest can't reproduce the problem is the test uses junit class loader which contains all the necessary url paths. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services