Vinay,
Nice idea, but we would have to get the BEEP or homegrown multiplexer
working first, yes?
- Paul
>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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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:[EMAIL PROTECTED]>
>>
>>>>For additional commands, e-mail:
>>>>
>>><mailto:[EMAIL PROTECTED]>
>>>
>>>>
>>>>
>>>
>>>
>>>
>>>--
>>>To unsubscribe, e-mail:
>>>
>><mailto:[EMAIL PROTECTED]>
>>
>>>For additional commands, e-mail:
>>><mailto:[EMAIL PROTECTED]>
>>>
>>
>>__________________________________________________
>>Do You Yahoo!?
>>Yahoo! Movies - coverage of the 74th Academy Awards®
>>http://movies.yahoo.com/
>>
>>--
>>To unsubscribe, e-mail:
>><mailto:[EMAIL PROTECTED]>
>>For additional commands, e-mail:
>><mailto:[EMAIL PROTECTED]>
>>
>
>
>__________________________________________________
>Do You Yahoo!?
>Yahoo! Movies - coverage of the 74th Academy Awards®
>http://movies.yahoo.com/
>
>--
>To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
>For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>
>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
