On Wed, Apr 9, 2008 at 1:14 AM,  <[EMAIL PROTECTED]> wrote:
> Author: viet
>  Date: Tue Apr  8 16:14:22 2008
>  New Revision: 646128
...
>  Modified: 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeBean.java
>  URL: 
> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeBean.java?rev=646128&r1=646127&r2=646128&view=diff
>  
> ==============================================================================
>  --- 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeBean.java
>  (original)
>  +++ 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeBean.java
>  Tue Apr  8 16:14:22 2008
>  @@ -25,6 +25,8 @@
>
>   import javax.persistence.PersistenceUnit;
>   import javax.ejb.EJBException;
>  +import javax.ejb.Local;
>  +import javax.ejb.Remote;
>   import javax.ejb.Stateless;
>   import javax.naming.Context;
>   import javax.naming.InitialContext;
>  @@ -32,6 +34,8 @@
>   import javax.persistence.EntityManager;
>   import javax.persistence.EntityManagerFactory;
>
>  [EMAIL PROTECTED](BankManagerFacadeRemote.class)
>  [EMAIL PROTECTED](BankManagerFacadeLocal.class)
>   @Stateless
>   public class BankManagerFacadeBean implements BankManagerFacadeLocal {

I can't find it in the spec, but may a bean specify a remote intf that
it doesn't implement? I think it's a bug in openejb if it allows it.
In such a single-interface case, mere @Local would be enough.

>
>
>  Modified: 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeLocal.java
>  URL: 
> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeLocal.java?rev=646128&r1=646127&r2=646128&view=diff
>  
> ==============================================================================
>  --- 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeLocal.java
>  (original)
>  +++ 
> geronimo/samples/trunk/samples/bank/bank-ejb/src/main/java/org/apache/geronimo/samples/bank/ejb/BankManagerFacadeLocal.java
>  Tue Apr  8 16:14:22 2008
>  @@ -17,10 +17,13 @@
>
>   package org.apache.geronimo.samples.bank.ejb;
>
>  +import javax.ejb.Local;
>  +
>   import org.apache.geronimo.samples.bank.ejb.Account;
>   import org.apache.geronimo.samples.bank.ejb.ExchangeRate;
>   import org.apache.geronimo.samples.bank.ejb.Customer;
>
>  [EMAIL PROTECTED]
>   public interface BankManagerFacadeLocal {
>         public java.util.List<Account> getAccountInformation(String 
> customerId);

Why is it needed? You can leave it out and let a bean specify whether
it's a local or remote interface (regardless of the interface's name).
If it's to show people it works I'm fine with it, otherwise I'm
puzzled ;-)

Jacek

-- 
Jacek Laskowski
http://www.JacekLaskowski.pl

Reply via email to