[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