[GitHub] tomee pull request #172: [7.0.x][Tomee-2243] jmx on managed executors

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] tomee pull request #172: [7.0.x][Tomee-2243] jmx on managed executors

deki
GitHub user exabrial opened a pull request:

    https://github.com/apache/tomee/pull/172

    [7.0.x][Tomee-2243] jmx on managed executors

    Expose some details about the underlying thread pool via JMX.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/exabrial/tomee issues/TOMEE-2243_jmx-on-managed-executors

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tomee/pull/172.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #172
   
----
commit 1590538beb144aacc576728bb6c34512e05a47a8
Author: Jonathan S. Fisher <exabrial@...>
Date:   2018-09-24T03:44:17Z

    Allow for JMX on ManagedExecutorServices

----


---
Reply | Threaded
Open this post in threaded view
|

Re: [GitHub] tomee pull request #172: [7.0.x][Tomee-2243] jmx on managed executors

exabrial12
We're running this patch in staging along with JGallimore's JMX patch[es].
It's humming along beautifully, we'd appreciate a merge!

Here's a visualization from Chronograf:
<http://tomee-openejb.979440.n4.nabble.com/file/t374660/Screen_Shot_2018-09-28_at_11.png>



--
Sent from: http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html
Reply | Threaded
Open this post in threaded view
|

Re: [GitHub] tomee pull request #172: [7.0.x][Tomee-2243] jmx on managed executors

exabrial12
Hey guys, can this PR get merged into 7.0.x, 7.1.x and 8.0.x? Thanks!



--
Sent from: http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html
Reply | Threaded
Open this post in threaded view
|

[GitHub] tomee pull request #172: [7.0.x][Tomee-2243] jmx on managed executors

deki
In reply to this post by deki
Github user exabrial closed the pull request at:

    https://github.com/apache/tomee/pull/172


---