[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-29 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/684 At this point, this is the best solution to ensure that it works across different impls and spec jars. +1 --- If your project is set up for it, you can reply to this email and have your

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/684 @johnament I wouldn't need this whole thing if this was merged. Just sent the PR: https://github.com/apache/geronimo-specs/pull/4 I"m not sure about the

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/684 Mmmph. So I can fix the spec jar for geronimo, but the problem will persist in the other spec jars, e.g. the same problem happens w/ the RI JAR and the JBoss JAR. Minor

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/684 Will wait for you then. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/684 i thought about changing JsonProvider to cache the loaded so this wouldn't be an issue (I could load it under the right class loader once) --- If your project is set up for it, you

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/684 If you don't mind, let me take a look over the weekend. This sounds like more of an issue with the geronimo spec jar, which we can get changes into.

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/684 If you build and run message-count example without this fix you will see a class not found exception. The server is not loaded worth the appClassLoader (system class loader).

[GitHub] activemq-artemis issue #684: ARTEMIS-565 Dealing with ClassLoading issues on...

2016-07-28 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/684 What was happening prior to this change? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have