ARTEMIS-5427 Integrate Jolokia's MBeanInfo cache

[jbertram]

The value of jolokia.version will need to be changed once jolokia/jolokia#925 is merged and released. Leaving this as a draft until then.

This update makes a significant difference in high-load situations (e.g. thousands of addresses & queues) for the following:

• Network transfer of JSON payload
• CPU utilization on the broker
• Memory consumption on the broker

I believe this will address the problems users have been reporting with web console load times and OOME on the broker.

[jbertram]

Jolokia releases are done. This should be ready to merge.

[gemmellr]

With so many deps, and potential for others appearing later and then not being managed, maybe using the bom would be better?

[jbertram]

I thought the same thing at first so I tried using:

            <dependency>
                <groupId>org.jolokia</groupId>
                <artifactId>jolokia-bom</artifactId>
                <version>${jolokia.version}</version>
            </dependency>

However, all of the Jolokia-related jars in the WAR aside from the one for jolokia-server-detector came from 2.3.0. Maybe I'm missing something?

[jbertram]

Ignore my previous comment. I think I've figured it out. Push incoming.

[gemmellr]

That snippet only manages the version of the artifact, if you were to have a unversioned/transitive dependency on it elsewhere in the build. You need to set the type to pom and scope to import to use the contained dependency entries as dependencyManagement, i.e to actually use it as a bom. Something like:

            <dependency>
                <groupId>org.jolokia</groupId>
                <artifactId>jolokia-bom</artifactId>
                <version>${jolokia.version}</version>
                <type>pom</type>
                <scope>import</scope>
            </dependency>

[gemmellr]

Hmm, view was stale, didnt see your comment.

[jbertram]

Ninja'd!

[GChuf]

Testing latest console build, I can only see 1 call to jolokia that has "listCache=true" parameter - am I missing something?
The request with the listCache=true:
{"type":"list","path":"hawtio/type=security,name=RBACRegistry"}