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

[rmannibucau]

Github user rmannibucau commented on a diff in the pull request:

    https://github.com/apache/tomee/pull/175#discussion_r221449894
  
    --- Diff: 
container/openejb-jpa-integration/src/main/java/org/apache/openejb/jpa/integration/eclipselink/OpenEJBServerPlatform.java
 ---
    @@ -63,4 +65,11 @@ protected void 
registerSynchronization_impl(AbstractSynchronizationListener list
                 transaction.registerInterposedSynchronization(synchronization);
             }
         }
    +
    +        @Override
    +        public void prepareServerSpecificServicesMBean() {
    +            if (getDatabaseSession() != null && shouldRegisterRuntimeBean) 
{
    +                 this.setRuntimeServicesMBean(new 
MBeanOpenEJBRuntimeServices(getDatabaseSession()));
    --- End diff --
    
    ensure it uses mBeanServer which has a toggle "skip jmx" on the instance


---