[jboss-cvs] JBossAS SVN: r67512 - in projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss: ejb3 and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: ALRubinger
Date: 2007-11-27 16:24:26 -0500 (Tue, 27 Nov 2007)
New Revision: 67512

Added:
   projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/defaults/
Removed:
   projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/ProxyFactory.java
   projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/remoting/
Modified:
   projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/Pool.java
   projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/RemoteBinding.java
Log:
Resolved API Leakage for RemoteProxyFactory in @RemoteBinding

Modified: projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/Pool.java
===================================================================
--- projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/Pool.java	2007-11-27 21:18:47 UTC (rev 67511)
+++ projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/Pool.java	2007-11-27 21:24:26 UTC (rev 67512)
@@ -26,6 +26,8 @@
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
+import org.jboss.ejb3.defaults.PoolDefaults;
+
 /**
  * Annotation for specifying the pool class to manage thread pooling for a bean
  *
@@ -36,7 +38,7 @@
 @Target({ElementType.TYPE})
 public @interface Pool
 {
-   String value();
-   int maxSize() default 30;
+   String value() default PoolDefaults.POOL_IMPLEMENTATION_THREADLOCAL;
+   int maxSize() default PoolDefaults.DEFAULT_POOL_SIZE;
    long timeout()default Long.MAX_VALUE;
 }

Modified: projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/RemoteBinding.java
===================================================================
--- projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/RemoteBinding.java	2007-11-27 21:18:47 UTC (rev 67511)
+++ projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/annotation/ejb/RemoteBinding.java	2007-11-27 21:24:26 UTC (rev 67512)
@@ -26,7 +26,7 @@
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
-import org.jboss.ejb3.remoting.RemoteProxyFactory;
+import org.jboss.ejb3.defaults.RemoteBindingDefaults;
 
 /**
  * Annotation for specifying the remote jndi binding for a bean as well
@@ -37,15 +37,15 @@
  * @author <a href="mailto:bill at jboss.org">Bill Burke</a>
  * @version $Revision: 64133 $
  */
- at Target({ElementType.TYPE, ElementType.ANNOTATION_TYPE})
+ at Target(
+{ElementType.TYPE, ElementType.ANNOTATION_TYPE})
 @Retention(RetentionPolicy.RUNTIME)
-public @interface RemoteBinding
-{
+public @interface RemoteBinding {
    String jndiBinding() default "";
 
    String interceptorStack() default "";
 
    String clientBindUrl() default "";
 
-   Class<? extends RemoteProxyFactory> factory() default RemoteProxyFactory.class;
+   String factory() default RemoteBindingDefaults.PROXY_FACTORY_DEFAULT;
 }

Deleted: projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/ProxyFactory.java
===================================================================
--- projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/ProxyFactory.java	2007-11-27 21:18:47 UTC (rev 67511)
+++ projects/ejb3/trunk/ejb3-ext-api/src/main/java/org/jboss/ejb3/ProxyFactory.java	2007-11-27 21:24:26 UTC (rev 67512)
@@ -1,41 +0,0 @@
-/*
- * JBoss, Home of Professional Open Source.
- * Copyright 2006, Red Hat Middleware LLC, and individual contributors
- * as indicated by the @author tags. See the copyright.txt file in the
- * distribution for a full listing of individual contributors.
- *
- * This is free software; you can redistribute it and/or modify it
- * under the terms of the GNU Lesser General Public License as
- * published by the Free Software Foundation; either version 2.1 of
- * the License, or (at your option) any later version.
- *
- * This software is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this software; if not, write to the Free
- * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
- * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
- */
-package org.jboss.ejb3;
-
-/**
- * Comment
- *
- * @author <a href="mailto:bill at jboss.org">Bill Burke</a>
- * @version $Revision: 64133 $
- */
-public interface ProxyFactory
-{
-   public Object createHomeProxy();
-   
-   public Object createProxy();
-
-   public Object createProxy(Object id);
-   
-   public void start() throws Exception;
-
-   public void stop() throws Exception;
-}