cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jervis Liu" <jervis...@gmail.com>
Subject Re: Patch for JAX-RS Spring configuration [CXF-1416]
Date Fri, 08 Feb 2008 01:43:49 GMT
I ll do this.

Cheers,
Jervis

On Feb 8, 2008 1:15 AM, Barry Fitzgerald <barfitzgerald@gmail.com> wrote:

> I've added a patch to Jira now that includes a test. Both patches still
> need
> to applied - one for the code, one for the test.
>
> Thanks,
>
> Barry
>
> On Feb 7, 2008 10:09 AM, Beryozkin, Sergey <Sergey.Beryozkin@iona.com>
> wrote:
>
> > Hi Barry
> >
> > Thanks for the explanation. I've also read the Jervis's comments, so it
> > does make sense. It's useful to define multiple (root) resources
> > possibly sharing the same configuration.
> > I had a minor concern, which was purely about the syntax :
> >
> > > > <jaxrs:server id="customerService" address="/">
> > > >        <jaxrs:serviceBeans>
> > > >            <!--multiple services support -->
> > > >            <ref bean="restfulUserService" />
> > > >            <ref bean="restfulLoginService" />
> > > >        </jaxrs:serviceBeans>
> > > >    </jaxrs:server>
> >
> > It just says me that a customer service is a master service.
> > But then, may be, this is exactly what a user wants to express, to show
> > this sort of relationship...
> >
> > Cheers, Sergey
> >
> > -----Original Message-----
> > From: Barry Fitzgerald [mailto:barfitzgerald@gmail.com]
> > Sent: 06 February 2008 21:05
> > To: cxf-dev@incubator.apache.org
> > Subject: Re: Patch for JAX-RS Spring configuration [CXF-1416]
> >
> > Hi Sergey,
> >
> > Thanks for the comment. My rational is as follows:
> >
> >   - The spring config method should closely mirror the non-spring
> > config
> >   method.
> >   - There may be many situations where your server is customised in
> > some
> >   way e.g. injected interceptors - I think in these cases it wouldn't
> >   make sense to have multiple servers defined each with the same
> >   customisation.
> >   - While some scenarios may be covered by sub-resources, in many this
> >   is not applicable.
> >   - Finally this the patch allows the choice to inject multiple root
> >   resources or not, allowing one to choose their ideal approach.
> >
> > In short it's no show stopper but I think it is useful functionality.
> >
> > Barry
> >
> > On Feb 6, 2008 4:13 PM, Sergey Beryozkin <sergey.beryozkin@iona.com>
> > wrote:
> >
> > > Hi
> > >
> > > So what sort of problem this patch  is trying to address ?
> > > Why can't a user do :
> > >
> > > <jaxrs:server id="customerService" address="/">
> > > </jaxrs:server>
> > >
> > > <jaxrs:server id="restfulUserService" address="/user">
> > > </jaxrs:server>
> > >
> > > The proposed configuration makes me think that, in this case, a
> > > restfulUserService is a subresource...
> > >
> > > Cheers, Sergey
> > >
> > > > Hi,
> > > >
> > > > I've created a patch for CXF-1416. It now enables multiple root
> > > resources to
> > > > be configured.
> > > >
> > > > https://issues.apache.org/jira/browse/CXF-1416
> > > >
> > > > I've tested this manually with the following similar config to
> > below:
> > > >
> > > > <jaxrs:server id="customerService" address="/">
> > > >        <jaxrs:serviceBeans>
> > > >            <!--multiple services support -->
> > > >            <ref bean="restfulUserService" />
> > > >            <ref bean="restfulLoginService" />
> > > >        </jaxrs:serviceBeans>
> > > >    </jaxrs:server>
> > > >
> > > > There are currently no tests for the Spring bean definition parser
> > that
> > > I
> > > > can see and I'm not sure exactly how to write a valid unit test for
> > > this.
> > > > Maybe someone can enlighten me!
> > > >
> > > > Thanks,
> > > >
> > > > Barry
> > > >
> > >
> > > ----------------------------
> > > IONA Technologies PLC (registered in Ireland)
> > > Registered Number: 171387
> > > Registered Address: The IONA Building, Shelbourne Road, Dublin 4,
> > Ireland
> > >
> >
> > ----------------------------
> > IONA Technologies PLC (registered in Ireland)
> > Registered Number: 171387
> > Registered Address: The IONA Building, Shelbourne Road, Dublin 4,
> Ireland
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message