You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/11/26 10:52:46 UTC

svn commit: r598194 - in /mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx: IoFilterMBean.java ObjectMBean.java

Author: trustin
Date: Mon Nov 26 01:52:45 2007
New Revision: 598194

URL: http://svn.apache.org/viewvc?rev=598194&view=rev
Log:
Fixed a problem that ThreadPoolExecutor.queue property is not excluded

Modified:
    mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/IoFilterMBean.java
    mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/ObjectMBean.java

Modified: mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/IoFilterMBean.java
URL: http://svn.apache.org/viewvc/mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/IoFilterMBean.java?rev=598194&r1=598193&r2=598194&view=diff
==============================================================================
--- mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/IoFilterMBean.java (original)
+++ mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/IoFilterMBean.java Mon Nov 26 01:52:45 2007
@@ -18,7 +18,6 @@
 
 import org.apache.mina.common.IoFilter;
 import org.apache.mina.common.IoSession;
-import org.apache.mina.filter.executor.ExecutorFilter;
 
 /**
  * A JMX MBean wrapper for an {@link IoSession}.
@@ -32,15 +31,6 @@
         super(source);
     }
     
-    @Override
-    protected boolean isReadable(Class<?> type, String attrName) {
-        if (ExecutorFilter.class.isAssignableFrom(type) &&
-            attrName.equals("executor.queue")) {
-            return false;
-        }
-        return super.isReadable(type, attrName);
-    }
-
     @Override
     protected boolean isOperation(String methodName) {
         // Ignore some IoFilter methods.

Modified: mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/ObjectMBean.java
URL: http://svn.apache.org/viewvc/mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/ObjectMBean.java?rev=598194&r1=598193&r2=598194&view=diff
==============================================================================
--- mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/ObjectMBean.java (original)
+++ mina/trunk/integration-jmx/src/main/java/org/apache/mina/integration/jmx/ObjectMBean.java Mon Nov 26 01:52:45 2007
@@ -532,6 +532,10 @@
             return false;
         }
         
+        if (ThreadPoolExecutor.class.isAssignableFrom(type) && attrName.equals("queue")) {
+            return false;
+        }
+
         return true;
     }