[GitHub] tomee issue #170: [7.0.x Branch] TOMEE-2240 ManagedScheduledExecutorService ...

2018-09-27 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/170 > While this class inherits from ThreadPoolExecutor, a few of the inherited tuning methods are not useful for it. In particular, because it acts as a fixed-sized pool using corePoolSize thre

[GitHub] tomee pull request #170: [7.0.x Branch] TOMEE-2240 ManagedScheduledExecutorS...

2018-09-27 Thread exabrial
Github user exabrial closed the pull request at: https://github.com/apache/tomee/pull/170 ---

[GitHub] tomee issue #166: Apply Jonathan Gallimore's JMX Patch to 7.0.5

2018-09-27 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/166 It looks like on the ActiveMQ resource adapter, `mBeanInfo.getAttributes()` is returning null, breaking hawtio. They should obviously handle this case, but also guessing we're probably not supposed

[GitHub] tomee issue #166: Apply Jonathan Gallimore's JMX Patch to 7.0.5

2018-09-27 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/166 Thanks for the fix! I'm surprised the jvm allowed the nulls. ---

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

2018-09-27 Thread exabrial
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] tomee pull request #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on in...

2018-10-08 Thread exabrial
Github user exabrial commented on a diff in the pull request: https://github.com/apache/tomee/pull/175#discussion_r223455236 --- Diff: container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/MBeanOpenEJBRuntimeServices.java --- @@ -0,0 +1,26

[GitHub] tomee pull request #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on in...

2018-10-08 Thread exabrial
Github user exabrial commented on a diff in the pull request: https://github.com/apache/tomee/pull/175#discussion_r223456172 --- Diff: container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java --- @@ -63,4 +65,11

[GitHub] tomee issue #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on init

2018-10-08 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/175 @rmannibucau If this looks good, could you merge? ---

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

2018-10-16 Thread exabrial
Github user exabrial closed the pull request at: https://github.com/apache/tomee/pull/172 ---

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

2018-10-27 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/179 [tomee-7.0.x] [Tomee-2243] jmx on managed executors You can merge this pull request into a Git repository by running: $ git pull https://github.com/exabrial/tomee exabrial/TOMEE-2243_jmx

[GitHub] tomee pull request #184: [tomee-7.1.x] [tomee-2268] upgrade cxf to 3.1.8

2018-11-03 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/184 [tomee-7.1.x] [tomee-2268] upgrade cxf to 3.1.8 [tomee-7.1.x] [tomee-2268] upgrade cxf to 3.1.8: https://bit.ly/2SGYGm6 You can merge this pull request into a Git repository by running: $ git

[GitHub] tomee pull request #182: [Master] [TOMEE-2269] Upgrade CXF to 3.2.7 per user...

2018-11-03 Thread exabrial
Github user exabrial closed the pull request at: https://github.com/apache/tomee/pull/182 ---

[GitHub] tomee pull request #183: [tomee-7.0.x] [TOMEE-2268] Upgrade CXF to 3.1.8per ...

2018-11-03 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/183 [tomee-7.0.x] [TOMEE-2268] Upgrade CXF to 3.1.8per user request: https://bit.ly/2SGYGm6 [tomee-7.0.x] [TOMEE-2268] Upgrade CXF to 3.1.8per user request: https://bit.ly/2SGYGm6 You can merge

[GitHub] tomee pull request #182: [Master] [TOMEE-2269] Upgrade CXF to 3.2.7 per user...

2018-11-03 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/182 [Master] [TOMEE-2269] Upgrade CXF to 3.2.7 per user request: https://bit.ly/2SGYGm6 [Master] [TOMEE-2269] Upgrade CXF to 3.2.7 per user request: https://bit.ly/2SGYGm6 You can merge this pull

[GitHub] tomee issue #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on init

2018-09-29 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/175 Fix TOMEE-2249 https://issues.apache.org/jira/projects/TOMEE/issues/TOMEE-2249?filter=reportedbyme by implementing required interfaces and executing method ---

[GitHub] tomee pull request #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on in...

2018-09-29 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/175 [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on init You can merge this pull request into a Git repository by running: $ git pull https://github.com/exabrial/tomee issues/TOMEE

[GitHub] tomee pull request #175: [tomee-7.0.x][TOMEE-2249] Fix EclipseLink NPE on in...

2018-10-08 Thread exabrial
Github user exabrial commented on a diff in the pull request: https://github.com/apache/tomee/pull/175#discussion_r223483193 --- Diff: container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java --- @@ -26,13 +27,14

[GitHub] tomee pull request #171: [Master Branch] TOMEE-2240 ManagedScheduledExecutor...

2018-09-26 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/171 [Master Branch] TOMEE-2240 ManagedScheduledExecutorService doesn't allow setting max value Fix https://issues.apache.org/jira/projects/TOMEE/issues/TOMEE-2240 You can merge this pull request

[GitHub] tomee pull request #170: TOMEE-2240 ManagedScheduledExecutorService doesn't ...

2018-09-26 Thread exabrial
GitHub user exabrial opened a pull request: https://github.com/apache/tomee/pull/170 TOMEE-2240 ManagedScheduledExecutorService doesn't allow setting max value Fix https://issues.apache.org/jira/projects/TOMEE/issues/TOMEE-2240 You can merge this pull request into a Git repository

[GitHub] tomee issue #171: [Master Branch] TOMEE-2240 ManagedScheduledExecutorService...

2018-09-26 Thread exabrial
Github user exabrial commented on the issue: https://github.com/apache/tomee/pull/171 Actually I need to close this! I wrote this patch before I read the javadoc on the scheduled executor service thoroughly. The max setting is a little misleading. When you wrote the original code you

[GitHub] tomee pull request #171: [Master Branch] TOMEE-2240 ManagedScheduledExecutor...

2018-09-26 Thread exabrial
Github user exabrial closed the pull request at: https://github.com/apache/tomee/pull/171 ---