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 Thu, 21 Mar 2002 22:41:27 GMT
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>


Mime
View raw message