Merge pull request #540 from smirzai/fixJmxLoggers

fixed logging and description typos in JMX module
flowable · Mar 10, 2015 · 715113f · 715113f
2 parents 2e8aa7b + 9bddb78
commit 715113f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/...tiviti-jmx/src/main/java/org/activiti/management/jmx/DefaultManagementMBeanAssembler.java b/...tiviti-jmx/src/main/java/org/activiti/management/jmx/DefaultManagementMBeanAssembler.java
@@ -28,7 +28,7 @@
  */
 public class DefaultManagementMBeanAssembler implements ManagementMBeanAssembler {
 
-  private static final Logger LOG = LoggerFactory.getLogger(DefaultManagementAgent.class);
+  private static final Logger LOG = LoggerFactory.getLogger(DefaultManagementMBeanAssembler.class);
 
   protected final MBeanInfoAssembler assembler;
 

diff --git a/modules/activiti-jmx/src/main/java/org/activiti/management/jmx/MBeanInfoAssembler.java b/modules/activiti-jmx/src/main/java/org/activiti/management/jmx/MBeanInfoAssembler.java
@@ -45,7 +45,7 @@
 
 public class MBeanInfoAssembler {
 
-  private static final Logger LOG = LoggerFactory.getLogger(DefaultManagementAgent.class);
+  private static final Logger LOG = LoggerFactory.getLogger(MBeanInfoAssembler.class);
 
   protected final WeakHashMap<Class< ? >, MBeanAttributesAndOperations> cache = new WeakHashMap<Class< ? >, MBeanAttributesAndOperations>(10);
 

diff --git a/modules/activiti-jmx/src/main/java/org/activiti/management/jmx/mbeans/JobExecutorMBean.java b/modules/activiti-jmx/src/main/java/org/activiti/management/jmx/mbeans/JobExecutorMBean.java
@@ -21,7 +21,7 @@
 /**
  * @author Saeid Mirzaei
  */
-@ManagedResource(description = "Process definition MBean")
+@ManagedResource(description = "Job executor MBean")
 public class JobExecutorMBean {
 
   JobExecutor jobExecutor;
@@ -34,7 +34,7 @@ public JobExecutorMBean(ProcessEngineConfiguration processEngineConfig) {
 
   @ManagedAttribute(description = "check if the job executor is activated")
   public boolean isJobExecutorActivated() {
-    return jobExecutor == null || jobExecutor.isActive();
+    return jobExecutor != null && jobExecutor.isActive();
   }
 
   @ManagedOperation(description = "set job executor activate")