[jboss-cvs] JBossCache/src/org/jboss/cache/loader/tcp ...
Brian Stansberry
brian.stansberry at jboss.com
Wed May 23 02:34:31 EDT 2007
User: bstansberry
Date: 07/05/23 02:34:31
Modified: src/org/jboss/cache/loader/tcp TcpCacheServerMBean.java
Log:
[JBCACHE-1053] Remove setCache(CacheSPI) from mbean interface as it makes the interace illegal
Revision Changes Path
1.8 +6 -2 JBossCache/src/org/jboss/cache/loader/tcp/TcpCacheServerMBean.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: TcpCacheServerMBean.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/loader/tcp/TcpCacheServerMBean.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -b -r1.7 -r1.8
--- TcpCacheServerMBean.java 22 May 2007 20:12:40 -0000 1.7
+++ TcpCacheServerMBean.java 23 May 2007 06:34:31 -0000 1.8
@@ -11,7 +11,7 @@
*
* @author Bela Ban
* @author Brian Stansberry
- * @version $Id: TcpCacheServerMBean.java,v 1.7 2007/05/22 20:12:40 bstansberry Exp $
+ * @version $Id: TcpCacheServerMBean.java,v 1.8 2007/05/23 06:34:31 bstansberry Exp $
*/
public interface TcpCacheServerMBean
{
@@ -37,7 +37,11 @@
Cache getCache();
- void setCache(CacheSPI cache);
+ // BES 2007/5/22 Don't expose this setter via MBean interface, as
+ // it's a different type from getter, which is invalid. This
+ // setter doesn't need to be exposed vai JMX; the CacheJmxWrapper
+ // is sufficient, and the setter is still there in the impl.
+ //void setCache(CacheSPI cache);
/**
* Allows {@link #setCache(CacheSPI) injection of the CacheSPI} via
More information about the jboss-cvs-commits
mailing list