commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vinaysahil chandran <sahilvi...@yahoo.com>
Subject Re: [Altrmi] PATCH Passing pf remote obj ' ref
Date Fri, 22 Mar 2002 00:56:08 GMT
Paul,
org.apache.commons.altrmi.client.impl.stream.StreamInvocationHandler
has synchronized handleInvocation(...) .
With little efforts I guess we can make this
class ThreadSafe and thus get rid of sync block ...

Maybe we can  support async' calls with the altrmi 
too ??? (todo item maybe)

Comments ...:-?

Regards,
Vinay
--- vinaysahil chandran <sahilvinay@yahoo.com> wrote:
> Paul,
> 
> I worked with a fresh CVS checkout and it 
> (pipeda from tests.xml)
>  target works well without any hiccups ..
> I wasn't able to reproduce it out at my machine .
> Morevoer the patch's that I sent second time 
> doesn't
> seems to be commited(Passing_Facade.patch).
> Can you verify the way's I have employed there ???
> Basically,
>  I have tried to pass Remote ref wrapped inside
> FacadeREfHolder obj back to the Server.
> ( I think had written FacadeRefHolder for this 
> purpose but missed out on using  it to wrap
> arguments 
> sent  from the client.I have just written the 
> function to wrapp the Facade with
> FacadeRefHolder at the client side .)
> 
> I hope I have conveyed the stuff I did over the
> past 2  patchs well across to you...
> 
> Regards,
> V i n a y
> 
> --- Paul Hammant <Paul_Hammant@yahoo.com> wrote:
> > Vinay,
> > 
> > I have applied that too, and it is looking good.
> > One problem, test 'pipeda' fails now with a NPE at
> > this line in correct 
> > args :
> > 
> >     args[i] =
> > asih.mRefBeans.get(frh.getReferenceID());
> > 
> > Can you take a look dude..?
> > 
> > Regards,
> > 
> > - Paul
> > 
> > >Paul,
> > >No excuses , I had NOT done *ant clean* 
> > >before  building the src after the changes ..
> > >Here is the patch that I  missed ...
> > >
> > >But can you get the reason behind the 
> > >getMethodInvocationHandler(MethodRequest
> mr,String
> > >objectName)  ..
> > >
> > >Wont a more generic call like :
> > >getMethodInvocationHandler(String puslishedname)
> > >be better .
> > >Comments ?
> > >
> > >Sorry for this glitch,
> > >Vinay
> > >
> > >
> > >
> > >Index: AbstractServer.java
> >
>
>===================================================================
> > >RCS file:
> >
>
>/home/cvspublic/jakarta-commons-sandbox/altrmi/src/java/org/apache/commons/altrmi/server/impl/AbstractServer.java,v
> > >retrieving revision 1.21
> > >diff -r1.21 AbstractServer.java
> > >231a232,244
> > >
> > >>   /**
> > >>     * Method getMethodInvocationHandler
> > >>     *
> > >>     *
> > >>     * @param publishedName
> > >>     *
> > >>     * @return
> > >>     *
> > >>     */
> > >>    public MethodInvocationHandler
> > >>
> > >getMethodInvocationHandler(String publishedName)
> {
> > >
> > >>        return
> > >>
> >
>
>mInovcationHandlerAdapter.getMethodInvocationHandler(publishedName);
> > >
> > >>    }
> > >>
> > >
> > >
> > >
> > >
> > >
> > >
> >
> >__________________________________________________
> > >Do You Yahoo!?
> > >Yahoo! Movies - coverage of the 74th Academy
> > Awards®
> > >http://movies.yahoo.com/
> > >
> > >--
> > >To unsubscribe, e-mail:  
> >
> <mailto:commons-dev-unsubscribe@jakarta.apache.org>
> > >For additional commands, e-mail:
> > <mailto:commons-dev-help@jakarta.apache.org>
> > >
> > >
> > >
> > 
> > 
> > 
> > 
> > --
> > To unsubscribe, e-mail:  
> >
> <mailto:commons-dev-unsubscribe@jakarta.apache.org>
> > For additional commands, e-mail:
> > <mailto:commons-dev-help@jakarta.apache.org>
> > 
> 
> 
> __________________________________________________
> Do You Yahoo!?
> Yahoo! Movies - coverage of the 74th Academy Awards®
> http://movies.yahoo.com/
> 
> --
> To unsubscribe, e-mail:  
> <mailto:commons-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail:
> <mailto:commons-dev-help@jakarta.apache.org>
> 


__________________________________________________
Do You Yahoo!?
Yahoo! Movies - coverage of the 74th Academy Awards®
http://movies.yahoo.com/

--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message