Mailing-List: contact deltaspike-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: deltaspike-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of gerhard.petracek@gmail.com
 designates 209.85.213.175 as permitted sender)
Received-SPF: pass (google.com: domain of gerhard.petracek@gmail.com
 designates 10.236.156.34 as permitted sender) client-ip=10.236.156.34;
MIME-Version: 1.0
In-Reply-To: <1330964239.51958.YahooMailNeo@web171403.mail.ir2.yahoo.com>
References: <1330946380.89788.YahooMailNeo@web171406.mail.ir2.yahoo.com>
 <CAGJtJfEn+BU_UQAM4QsKi+aSkJM48qz_3=1o2RWE=VyKgTQK_g@mail.gmail.com>
 <1330962295.22350.YahooMailNeo@web171405.mail.ir2.yahoo.com>
 <CAGJtJfE-Yw_h+NCTdGrAV2+np0L4H6dQ4Vpeg3wbFGOVTjO1zg@mail.gmail.com>
 <1330964239.51958.YahooMailNeo@web171403.mail.ir2.yahoo.com>
From: Gerhard Petracek <gerhard.petracek@gmail.com>
Date: Mon, 5 Mar 2012 17:24:12 +0100
Message-ID: 
 <CAGJtJfFdZX443UhHqosToA6qP9DkyiauD2+wBd4T8-c5RtSfDg@mail.gmail.com>
Subject: Re: Re : Re : ConfigResolver : adding @ConfigProperty injection ?
To: deltaspike-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=20cf303f6d227def9a04ba815c5d

--20cf303f6d227def9a04ba815c5d
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

hi adrian,

would be ok for me.

regards,
gerhard


2012/3/5 Adrian Gonzalez <adr_gonzalez@yahoo.fr>

> What about having both options ?
>
> 1. be able to use directly @ConfigProperty
> @Produces
> public LoginContext
> produceLoginContext(@ConfigProperty("loginConfigFile") String
> loginConfigFileName, @ConfigProperty("loginModuleName") String
> loginModuleName)
>   }
> }
> Handy when all config values are only used in a centralized class (e.g.
> MyAppConfig).
>
> 2. type safe config annotations
> @ConfigProperty(
>   name =3D "pool_size",
>   eager =3D true, //true is also the default value -> the value gets
> converted during the bootstrapping process
>   converter =3D StringToIntegerConverter.class
> )
> public @interface PoolSize {
> }
>
> ----- Mail original -----
> De : Gerhard Petracek <gerhard.petracek@gmail.com>
> =C0 : deltaspike-dev@incubator.apache.org
> Cc :
> Envoy=E9 le : Lundi 5 mars 2012 17h08
> Objet : Re: Re : ConfigResolver : adding @ConfigProperty injection ?
>
> hi adrian,
>
> @#1:
> it isn't the default value of the property.
> eager should be true by default -> the configured value gets converted
> during bootstrapping (if the value has an invalid format the bootstrappin=
g
> process fails).
> if eager is false, the configured value will be converted directly before
> the injection (e.g. for values stored in dynamic config-sources).
>
> @#2:
> afaik there is an upcoming jsr about it.
>
> regards,
> gerhard
>
>
>
> 2012/3/5 Adrian Gonzalez <adr_gonzalez@yahoo.fr>
>
> > Hi Gerhard,
> >
> > 1. didn't understood the meaning of eager attribute
> > If eager =3D default value, it should have been some integer value in y=
our
> > sample
> >
> >
> > 2. StringToIntegerConverter converter
> > Is there an existing conversion API (standard, in deltaspike, ...) ?
> > (otherwise we're gonna need one here - and end up with another conversi=
on
> > API ;) )
> >
> > Thanks !
> >
> > ----- Mail original -----
> > De : Gerhard Petracek <gerhard.petracek@gmail.com>
> > =C0 : deltaspike-dev@incubator.apache.org
> > Cc :
> > Envoy=E9 le : Lundi 5 mars 2012 12h57
> > Objet : Re: ConfigResolver : adding @ConfigProperty injection ?
> >
> > hi adrian,
> >
> > if we agree also on adding a bit more to allow e.g.:
> >
> > //...
> > @ConfigProperty(
> >   name =3D "pool_size",
> >   eager =3D true, //true is also the default value -> the value gets
> > converted during the bootstrapping process
> >   converter =3D StringToIntegerConverter.class
> > )
> > public @interface PoolSize
> > {
> > }
> >
> > @Inject
> > @PoolSize
> > private int configuredPoolSize;
> >
> > then i would vote +1
> > (for sure the details need further discussions.)
> >
> > regards,
> > gerhard
> >
> >
> >
> > 2012/3/5 Adrian Gonzalez <adr_gonzalez@yahoo.fr>
> >
> > > Hello,
> > >
> > > Deltaspike config module is based on ConfigResolver usage :
> > >   ConfigResolver.getPropertyValue("test")
> > >
> > >
> > > Wouldn't it be interesting to add on top of it some injection
> > > capacity ? (i.e. providing @ConfigProperty annotation)
> > >
> > > Sample usage [1] :
> > > @Produces
> > > public LoginContext
> > produceLoginContext(@ConfigProperty("loginConfigFile")
> > > String loginConfigFileName,
> > >
> >  @ConfigProperty("loginModuleName")
> > > String loginModuleName)
> > >    blabla
> > > }
> > >
> > > This approach is based on Antonio's petstore application - config cod=
e
> is
> > > available in [2]
> > >
> > > [1]
> > >
> >
> https://github.com/agoncal/agoncal-application-petstore-ee6/blob/master/s=
rc/main/java/org/agoncal/application/petstore/security/LoginContextProducer=
.java
> > >
> > > [2]
> > >
> >
> https://github.com/agoncal/agoncal-application-petstore-ee6/blob/master/s=
rc/main/java/org/agoncal/application/petstore/util/ConfigProperty.java
> > >
> > >
> >
> https://github.com/agoncal/agoncal-application-petstore-ee6/blob/master/s=
rc/main/java/org/agoncal/application/petstore/util/ConfigPropertyProducer.j=
ava
> > >
> > >
> >
> >
>
>

--20cf303f6d227def9a04ba815c5d--