commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Hammant <Paul_Hamm...@yahoo.com>
Subject Re: [Altrmi] PATCH Passing pf remote obj ' ref
Date Fri, 22 Mar 2002 08:51:08 GMT
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 <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>
>
>
>




--
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