Fix rmannibucau concern on PR and explicity check for JMX runtime flag

Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/460ba29b
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/460ba29b
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/460ba29b

Branch: refs/heads/tomee-7.0.x
Commit: 460ba29b6dc0790e70b28920c47b04591d80a026
Parents: 515306f
Author: Jonathan S. Fisher <exabr...@gmail.com>
Authored: Mon Oct 8 13:21:17 2018 -0500
Committer: Jonathan S. Fisher <exabr...@gmail.com>
Committed: Mon Oct 8 13:21:17 2018 -0500

----------------------------------------------------------------------
 .../openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/460ba29b/container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
----------------------------------------------------------------------
diff --git 
a/container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
 
b/container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
index f68a962..cd73c26 100644
--- 
a/container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
+++ 
b/container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
@@ -68,7 +68,7 @@ public class OpenEJBServerPlatform extends 
JMXServerPlatformBase implements JMXE
 
         @Override
         public void prepareServerSpecificServicesMBean() {
-            if (getDatabaseSession() != null && shouldRegisterRuntimeBean) {
+            if (isRuntimeServicesEnabledDefault() && getDatabaseSession() != 
null && shouldRegisterRuntimeBean) {
                  this.setRuntimeServicesMBean(new 
MBeanOpenEJBRuntimeServices(getDatabaseSession()));
             }
         }

Reply via email to