Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cxf-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of sergey.beryozkin@iona.com
 designates 62.221.12.55 as permitted sender)
Message-ID: <00fb01c86a3e$1e141b90$e002050a@pcgroupiona.com>
From: "Sergey Beryozkin" <sergey.beryozkin@iona.com>
To: <cxf-dev@incubator.apache.org>
References: <6f5a2e920802061304t63d480d3o6449b06e68110e9f@mail.gmail.com>
 <7C9FE14C002B5D47BF50AE3B2C376FA1024F9C2D@emea-ems1.IONAGLOBAL.COM>
 <6f5a2e920802070915p336c42b0s514c935e153c922d@mail.gmail.com>
 <797beb000802071743w3b7cca08j20d98ce61804ac41@mail.gmail.com>
 <797beb000802071825m4d3386b1n5b393d38b2c0caf6@mail.gmail.com>
 <6f5a2e920802080222j55d5f853k31caa4d1267f6a45@mail.gmail.com>
Subject: Re: Patch for JAX-RS Spring configuration [CXF-1416]
Date: Fri, 8 Feb 2008 10:34:00 -0000
MIME-Version: 1.0
Content-Type: text/plain; format="flowed"; charset="iso-8859-1";
 reply-type="original"
Content-Transfer-Encoding: quoted-printable

Hi

use an 'Add To VersionControl' option. 'Create Patch' wil then pick it up.
If you do a 'Create Patch' form a trunk then a single patch incorporating a=
ll the chnages, in src and tests, in multiple modules,=20
will be created...

Cheers, Sergey

----- Original Message -----=20
From: "Barry Fitzgerald" <barfitzgerald@gmail.com>
To: <cxf-dev@incubator.apache.org>
Sent: Friday, February 08, 2008 10:22 AM
Subject: Re: Patch for JAX-RS Spring configuration [CXF-1416]


> Sorry about that it's attached now.
>
> Is there an easy way to do this?
>
> I.e.How should I create a patch when adding new files?
>
> Barry
>
> On Feb 8, 2008 2:25 AM, Jervis Liu <jervisliu@gmail.com> wrote:
>
>> Hi Barry, the PetStore class used by the system test is missing in your
>> patch. Could you please attach this file in Jira. Thanks.
>>
>> Cheers,
>> Jervis
>>
>> On Feb 8, 2008 9:43 AM, Jervis Liu <jervisliu@gmail.com> wrote:
>>
>> > 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.co=
m>
>> > > 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=3D"customerService" address=3D"/">
>> > > > > >        <jaxrs:serviceBeans>
>> > > > > >            <!--multiple services support -->
>> > > > > >            <ref bean=3D"restfulUserService" />
>> > > > > >            <ref bean=3D"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 i=
n
>> > > > 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 ro=
ot
>> > > >   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.co=
m>
>> > > > wrote:
>> > > >
>> > > > > Hi
>> > > > >
>> > > > > So what sort of problem this patch  is trying to address ?
>> > > > > Why can't a user do :
>> > > > >
>> > > > > <jaxrs:server id=3D"customerService" address=3D"/">
>> > > > > </jaxrs:server>
>> > > > >
>> > > > > <jaxrs:server id=3D"restfulUserService" address=3D"/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 roo=
t
>> > > > > 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=3D"customerService" address=3D"/">
>> > > > > >        <jaxrs:serviceBeans>
>> > > > > >            <!--multiple services support -->
>> > > > > >            <ref bean=3D"restfulUserService" />
>> > > > > >            <ref bean=3D"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 tes=
t
>> > > 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
>> > > >
>> > >
>> >
>> >
>>
>=20

----------------------------
IONA Technologies PLC (registered in Ireland)
Registered Number: 171387
Registered Address: The IONA Building, Shelbourne Road, Dublin 4, Ireland