Re: [JCS] Rework logging

[Yogesh Rao]

Hi Gary,

Not that i have anything against the log4j 2 guys :-)  its just that log4j
2 combines both the API and a implementation I am right now not sure of how
the segregation has been done between them i.e. during deployment when
using it in API mode does it have any dependency on other log4j2 jars etc.

Regards,
-Yogesh


On Wed, Aug 20, 2014 at 7:31 PM, Gary Gregory 
wrote:

> On Wed, Aug 20, 2014 at 9:54 AM, Yogesh Rao  wrote:
>
> > Though i am not a member of the dev team :-) I would support a full
> fledge
> > facade implementation which doesn't provide any logging at all and let
> the
> > framework user decide which logging would he would like to bind it to.
> > SLF4J does that very neatly and its also easy, i am also aware it might
> > turn down the performance a little.
> >
>
> Log4J 2 also provides an API, see
> https://logging.apache.org/log4j/2.x/manual/api.html
>
> Gary
>
>
> >
> > Regards,
> > -Yogesh
> >
> >
> > On Wed, Aug 20, 2014 at 7:17 PM, sebb  wrote:
> >
> > > On 20 August 2014 14:37, Gary Gregory  wrote:
> > > > On Wed, Aug 20, 2014 at 9:28 AM, sebb  wrote:
> > > >
> > > >> On 20 August 2014 14:04, Gary Gregory 
> wrote:
> > > >> > Moving discussion about logging from [JCS-122] to this dev ML.
> > > >> >
> > > >> > Why not use Log4j 2, uses can redirect logging to other frameworks
> > if
> > > >> > needed.
> > > >>
> > > >> Why not use Commons Logging, can redirect logging to other
> frameworks
> > if
> > > >> needed?
> > > >>
> > > >
> > > > I'd like to think that Commons Logging has been deprecated by Log4j 2
> > can
> > >
> > > That is not an opinion that is universally shared.
> > >
> > > > do the same thing (in principle) AND provide it's own advanced
> logging
> > > > framework.
> > >
> > > s/it's/its/
> > >
> > > That may be so, but I don't think that is sufficient reason to choose
> > > Log4j2 over any other library.
> > >
> > > > Gary
> > > >
> > > >
> > > >> > Gary
> > > >> >
> > > >> > --
> > > >> > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> > > >> > Java Persistence with Hibernate, Second Edition
> > > >> > <http://www.manning.com/bauer3/>
> > > >> > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/
> >
> > > >> > Spring Batch in Action <http://www.manning.com/templier/>
> > > >> > Blog: http://garygregory.wordpress.com
> > > >> > Home: http://garygregory.com/
> > > >> > Tweet! http://twitter.com/GaryGregory
> > > >>
> > > >>
> -
> > > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > > >> For additional commands, e-mail: dev-h...@commons.apache.org
> > > >>
> > > >>
> > > >
> > > >
> > > > --
> > > > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> > > > Java Persistence with Hibernate, Second Edition
> > > > <http://www.manning.com/bauer3/>
> > > > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
> > > > Spring Batch in Action <http://www.manning.com/templier/>
> > > > Blog: http://garygregory.wordpress.com
> > > > Home: http://garygregory.com/
> > > > Tweet! http://twitter.com/GaryGregory
> > >
> > > -
> > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > > For additional commands, e-mail: dev-h...@commons.apache.org
> > >
> > >
> >
>
>
>
> --
> E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> Java Persistence with Hibernate, Second Edition
> <http://www.manning.com/bauer3/>
> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
> Spring Batch in Action <http://www.manning.com/templier/>
> Blog: http://garygregory.wordpress.com
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory
>

Re: [JCS] Rework logging

[Yogesh Rao]

Though i am not a member of the dev team :-) I would support a full fledge
facade implementation which doesn't provide any logging at all and let the
framework user decide which logging would he would like to bind it to.
SLF4J does that very neatly and its also easy, i am also aware it might
turn down the performance a little.

Regards,
-Yogesh


On Wed, Aug 20, 2014 at 7:17 PM, sebb  wrote:

> On 20 August 2014 14:37, Gary Gregory  wrote:
> > On Wed, Aug 20, 2014 at 9:28 AM, sebb  wrote:
> >
> >> On 20 August 2014 14:04, Gary Gregory  wrote:
> >> > Moving discussion about logging from [JCS-122] to this dev ML.
> >> >
> >> > Why not use Log4j 2, uses can redirect logging to other frameworks if
> >> > needed.
> >>
> >> Why not use Commons Logging, can redirect logging to other frameworks if
> >> needed?
> >>
> >
> > I'd like to think that Commons Logging has been deprecated by Log4j 2 can
>
> That is not an opinion that is universally shared.
>
> > do the same thing (in principle) AND provide it's own advanced logging
> > framework.
>
> s/it's/its/
>
> That may be so, but I don't think that is sufficient reason to choose
> Log4j2 over any other library.
>
> > Gary
> >
> >
> >> > Gary
> >> >
> >> > --
> >> > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> >> > Java Persistence with Hibernate, Second Edition
> >> > 
> >> > JUnit in Action, Second Edition 
> >> > Spring Batch in Action 
> >> > Blog: http://garygregory.wordpress.com
> >> > Home: http://garygregory.com/
> >> > Tweet! http://twitter.com/GaryGregory
> >>
> >> -
> >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >> For additional commands, e-mail: dev-h...@commons.apache.org
> >>
> >>
> >
> >
> > --
> > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> > Java Persistence with Hibernate, Second Edition
> > 
> > JUnit in Action, Second Edition 
> > Spring Batch in Action 
> > Blog: http://garygregory.wordpress.com
> > Home: http://garygregory.com/
> > Tweet! http://twitter.com/GaryGregory
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

Re: [beanutils2] SANDBOX-472

[Yogesh Rao]

Hi Benedikt,

Have attached the updated patch in JIRA. Please do apply it if everything
is found okay.

Regards,
-Yogesh


On Wed, Jul 23, 2014 at 11:18 AM, Benedikt Ritter 
wrote:

> Hello Yogesh,
>
>
> 2014-07-22 16:42 GMT+02:00 Yogesh Rao :
>
> > Hello Benedikt,
> >
> > I have attached few files with their test cases which can be applied to
> the
> > BU2 trunk if everything is okay :-)
> >
> > I am working on the rest...
> >
>
> Great to hear from you! I'll need some time to look into your patches. I
> hope to get to it this weekend. Stay tuned!
>
> Benedikt
>
>
> >
> > Regards,
> > -Yogesh
> >
> >
> > On Mon, Jun 30, 2014 at 8:26 AM, Yogesh Rao  wrote:
> >
> > > Hi benedikt,
> > >
> > > No problem...and yes very much available for development..do let me
> know
> > > what you think on the comment...
> > >
> > > Regards
> > >
> > >
> > > On Monday, June 30, 2014, Benedikt Ritter  wrote:
> > >
> > >> Yogesh,
> > >>
> > >> sorry it took so long! I had little time for OSS lately... I hope
> you're
> > >> still up for some BU2 development ;-)
> > >>
> > >> Best regards,
> > >> Benedikt
> > >>
> > >>
> > >> 2014-06-09 4:44 GMT+02:00 Yogesh Rao :
> > >>
> > >> > Hi,
> > >> >
> > >> > I have updated the thread in JIRA with some of my thoughts ... let
> me
> > >> know
> > >> > if it aligns so i can help contribute to the project.
> > >> >
> > >> > Regards,
> > >> > -Yogesh
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> http://people.apache.org/~britter/
> > >> http://www.systemoutprintln.de/
> > >> http://twitter.com/BenediktRitter
> > >> http://github.com/britter
> > >>
> > >
> >
>
>
>
> --
> http://people.apache.org/~britter/
> http://www.systemoutprintln.de/
> http://twitter.com/BenediktRitter
> http://github.com/britter
>

Re: [beanutils2] SANDBOX-472

[Yogesh Rao]

Hello Benedikt,

I have attached few files with their test cases which can be applied to the
BU2 trunk if everything is okay :-)

I am working on the rest...

Regards,
-Yogesh


On Mon, Jun 30, 2014 at 8:26 AM, Yogesh Rao  wrote:

> Hi benedikt,
>
> No problem...and yes very much available for development..do let me know
> what you think on the comment...
>
> Regards
>
>
> On Monday, June 30, 2014, Benedikt Ritter  wrote:
>
>> Yogesh,
>>
>> sorry it took so long! I had little time for OSS lately... I hope you're
>> still up for some BU2 development ;-)
>>
>> Best regards,
>> Benedikt
>>
>>
>> 2014-06-09 4:44 GMT+02:00 Yogesh Rao :
>>
>> > Hi,
>> >
>> > I have updated the thread in JIRA with some of my thoughts ... let me
>> know
>> > if it aligns so i can help contribute to the project.
>> >
>> > Regards,
>> > -Yogesh
>> >
>>
>>
>>
>> --
>> http://people.apache.org/~britter/
>> http://www.systemoutprintln.de/
>> http://twitter.com/BenediktRitter
>> http://github.com/britter
>>
>

Re: [beanutils2] SANDBOX-472

[Yogesh Rao]

Hi benedikt,

No problem...and yes very much available for development..do let me know
what you think on the comment...

Regards

On Monday, June 30, 2014, Benedikt Ritter  wrote:

> Yogesh,
>
> sorry it took so long! I had little time for OSS lately... I hope you're
> still up for some BU2 development ;-)
>
> Best regards,
> Benedikt
>
>
> 2014-06-09 4:44 GMT+02:00 Yogesh Rao >:
>
> > Hi,
> >
> > I have updated the thread in JIRA with some of my thoughts ... let me
> know
> > if it aligns so i can help contribute to the project.
> >
> > Regards,
> > -Yogesh
> >
>
>
>
> --
> http://people.apache.org/~britter/
> http://www.systemoutprintln.de/
> http://twitter.com/BenediktRitter
> http://github.com/britter
>

[beanutils2] SANDBOX-472

[Yogesh Rao]

Hi,

I have updated the thread in JIRA with some of my thoughts ... let me know
if it aligns so i can help contribute to the project.

Regards,
-Yogesh

Re: [SANDBOX][BEANUTILS2] Property expressions

[Yogesh Rao]

How about giving the user the options

1. go the traditional way of calling the series of get methods and perform
the operation
or
2. the pure expression way (this wouldnt allow the user to mix with the
first style)

the 2nd option does seem very similar to what jxpath does for living, but
having said this i am not very sure of all the feature it comes bundled
with. Does it handle custom converters aswell?

Regards,
-Yogesh



On Fri, May 2, 2014 at 11:24 PM, Gary Gregory wrote:

> Nah, I'm just talking around the water cooler,  thinking aloud...
>
> Gary
>
>  Original message From: Benedikt Ritter <
> brit...@apache.org> Date:05/02/2014  12:59  (GMT-05:00)
> To: Commons Developers List 
> Subject: Re: [SANDBOX][BEANUTILS2] Property expressions
> 
> So you're saying we should leave this out of BU2?
>
>
> 2014-05-02 12:55 GMT+02:00 Gary Gregory :
>
> > There is also our own jxpath.
> >
> > Gary
> >
> >  Original message From: Benedikt Ritter <
> > brit...@apache.org> Date:05/02/2014  06:40  (GMT-05:00)
> > To: Commons Developers List 
> > Subject: [SANDBOX][BEANUTILS2] Property expressions
> 
> > Hi all,
> >
> > one of the goal of BeanUtils2 is to provide the same functionality as
> > BeanUtils1. In BeanUtils1 you can do something like this:
> >
> > BeanUtils.getProperty(person, "address.city.zipCode");
> >
> > This would be translated into:
> >
> > person.getAddress().getCity().getZipCode();
> >
> > The same can be done with mapped and indexed properties:
> >
> > BeanUtils.getProperty(person, "contact(5).name");
> >
> > which would be translated to:
> >
> > person.getContact(5).getName();
> >
> > BeanUtils2 provides a fluent API where the first example would be done
> via:
> >
> > on(person).get("address").get("city").get("zipCode")
> >
> > and the second:
> >
> > on(person).getIndexed("contact").at(5).get("name")
> >
> > We are currently thinking about how we can implement the property
> > expressions. We are discussing this in SANDBOX-464 [1] and there is
> already
> > a patch. I'm currently unsure whether we should allow mixing up the
> fluent
> > API and property expressions. The contributor makes some good examples of
> > what kind of awful code could be created:
> >
> > on(addressBook).get("provider(google).contact[5]
> >
> ").getMapped("address").with("home").get("street.yetAnotherNestedProperty")
> >
> > on the other hand I don't want to force people into doing stuff like
> this:
> >
> > String path = ...int pos = ...
> > on(bean).get(path + "[" + pos + "]");
> >
> >
> > Currently I'm tempted to allow mixing up both API styles, but I'd like to
> > here your opinion first.
> >
> > Benedikt
> >
> > [1] https://issues.apache.org/jira/browse/SANDBOX-464
> >
> > --
> > http://people.apache.org/~britter/
> > http://www.systemoutprintln.de/
> > http://twitter.com/BenediktRitter
> > http://github.com/britter
> >
>
>
>
> --
> http://people.apache.org/~britter/
> http://www.systemoutprintln.de/
> http://twitter.com/BenediktRitter
> http://github.com/britter
>

Re: [SANDBOX][BEANUTILS2] Changing code style

[Yogesh Rao]

Hi,

Eclipse style seems to be better :)

Regards,
-Yogesh


On Sat, May 3, 2014 at 7:42 AM, sebb  wrote:

> +1
>
> I don't like the
> curly brace
> on new line
> rule
> as it
> wastes vertical
> space
> !
>
>
> On 2 May 2014 14:57, Gary Gregory  wrote:
> > +1
> >
> > Gary
> >
> >
> > On Fri, May 2, 2014 at 6:53 AM, Benedikt Ritter 
> wrote:
> >
> >> Hi all,
> >>
> >> BeanUtils2 uses the maven code style rules. Every now and then I have to
> >> reject patches because they don't follow these conventions. I'd like to
> >> change the formatting rules to a more standard rule set for the
> following
> >> reasons:
> >>
> >> - maven style is very verbose, since it uses a lot of spaces and curly
> >> braces have to be placed on new lines
> >> - a great majority of our components uses the Sun/Eclipse formatting
> with
> >> spaces
> >> - Setting up IDEs like IntelliJ or Eclipse is easier with the
> Sun/Eclipse
> >> style, because the only thing you have to change is the tab policy.
> Using
> >> the maven style implies that one has to import the rule set.
> >>
> >> If no one objects, I'll reformat the source code and update the check
> style
> >> rules.
> >>
> >> Regards,
> >> Benedikt
> >>
> >>
> >> --
> >> http://people.apache.org/~britter/
> >> http://www.systemoutprintln.de/
> >> http://twitter.com/BenediktRitter
> >> http://github.com/britter
> >>
> >
> >
> >
> > --
> > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> > Java Persistence with Hibernate, Second Edition<
> http://www.manning.com/bauer3/>
> > JUnit in Action, Second Edition 
> > Spring Batch in Action 
> > Blog: http://garygregory.wordpress.com
> > Home: http://garygregory.com/
> > Tweet! http://twitter.com/GaryGregory
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

Re: [beanutils2] New JIRA Task added

[Yogesh Rao]

Hi,

I have added the patch to JIRA issue "SANDBOX-473"

Regards,
-Yogesh


On Fri, Apr 25, 2014 at 6:58 PM, Yogesh Rao  wrote:

> Hi Benedikt,
>
> I havent added patches yet wantes ro know if i need to add tht with jira
> created
>
> Regards,
> -Yogesh
>
>
> On Friday, April 25, 2014, Benedikt Ritter  wrote:
>
>> Hi Yogesh,
>>
>> thanks for your work on BU2. I'm a bit busy currently, that's why I
>> haven't
>> found the time to review your patches. I'll try to have a look tomorrow.
>>
>> Regards,
>> Benedikt
>>
>>
>> 2014-04-25 10:19 GMT+02:00 Yogesh Rao :
>>
>> > Hi,
>> >
>> > I have added 2 new tasks in JIRA :- SANDBOX-472 & SANDBOX-473
>> >
>> > Awaiting comments on the JIRA issues and also please let me know if i
>> need
>> > to attach patches right away to the issues
>> >
>> > Regards,
>> > -Yogesh
>> >
>>
>>
>>
>> --
>> http://people.apache.org/~britter/
>> http://www.systemoutprintln.de/
>> http://twitter.com/BenediktRitter
>> http://github.com/britter
>>
>

Re: [beanutils2] New JIRA Task added

[Yogesh Rao]

Hi Benedikt,

I havent added patches yet wantes ro know if i need to add tht with jira
created

Regards,
-Yogesh

On Friday, April 25, 2014, Benedikt Ritter  wrote:

> Hi Yogesh,
>
> thanks for your work on BU2. I'm a bit busy currently, that's why I haven't
> found the time to review your patches. I'll try to have a look tomorrow.
>
> Regards,
> Benedikt
>
>
> 2014-04-25 10:19 GMT+02:00 Yogesh Rao >:
>
> > Hi,
> >
> > I have added 2 new tasks in JIRA :- SANDBOX-472 & SANDBOX-473
> >
> > Awaiting comments on the JIRA issues and also please let me know if i
> need
> > to attach patches right away to the issues
> >
> > Regards,
> > -Yogesh
> >
>
>
>
> --
> http://people.apache.org/~britter/
> http://www.systemoutprintln.de/
> http://twitter.com/BenediktRitter
> http://github.com/britter
>

[beanutils2] New JIRA Task added

[Yogesh Rao]

Hi,

I have added 2 new tasks in JIRA :- SANDBOX-472 & SANDBOX-473

Awaiting comments on the JIRA issues and also please let me know if i need
to attach patches right away to the issues

Regards,
-Yogesh

[beanutils2] Automatic Datatype Transformers / Converters

[Yogesh Rao]

Hi,

I see that the datatype transformers still needs to be incorporated into
the BU2 project. I have a fair bit of idea on what needs to be done in that
i.e extending AbstractTransformer for the specific datatypes and
incorporating the call in DefaultBeanPropertySetter.invokeSetter method (is
my understanding right here?) so wanted to know if i have to contribute to
the source do i raise a JIRA for a specific datatype and proceed ahead
adding patches to it?

Please advice.

Regards,
-Yogesh

Re: [beanutils2] Checkstyle Errors

[Yogesh Rao]

Hello Benedikt,

Thanks for the reply ... I will not look at the checkstyle issues for
now... I looked at the mail pointed out earlier by you and will try to work
on it... I do understand now in a nutshell for BU2 to be released it needs
to have the exact same functionality what BU1 had and probably this could
be a starting point for me, However if possible please let me know apart
from the fluent API style coding design, are there any other features that
BU2 would cater?

Thanks Andre for those links they have been very helpful...

Regards,
-Yogesh


On Fri, Apr 4, 2014 at 3:17 PM, André Diermann wrote:

> Hi Yogesh,
>
> I had problems with the Maven codestyle when I started to contribute to BU2
> since most IDE does not support it by default. What helped me was [1] or
> [2] respectively. Just want to share this information with you in case who
> weren't aware already :-)
>
> [1] https://maven.apache.org/developers/conventions/code.html
> [2] https://maven.apache.org/developers/maven-eclipse-codestyle.xml
>
> Best regards,
> André
>
>
> 2014-04-04 10:40 GMT+02:00 Benedikt Ritter :
>
> > Hello Yogesh and welcome to the dev ML!
> >
> > Nice that you've decided to help out with the coding in [beanutils2]. I'm
> > not completely happy with the current check style configuration (which is
> > derived from the maven project). I'm planning to propose to switch to a
> > different config this is less verbose (longer lines, less line breaks).
> So
> > I wouldn't recommend getting started in this area. How ever there is a
> lot
> > to do in BU2. I've written what I think needs to be implemented a while
> ago
> > [1]. If you like, you can just start with one of the things I've pointed
> > out.
> >
> > Regards,
> > Benedikt
> >
> > [1] http://markmail.org/message/pv2qpr7em4ftnulj
> >
> >
> > 2014-04-03 16:36 GMT+02:00 Yogesh Rao :
> >
> > > Hi,
> > >
> > > I was planning to contribute to the bu2 project and thought fixing
> > > checkstyle errors should be a good start to it.
> > > I ran the mvn checkstyle:checkstyle command to generate the report
> which
> > > ended up showing 3218 errors. I wanted to know if all the errors needs
> to
> > > fixed or only few specific ones.
> > >
> > > Please suggest ..
> > >
> > > Cheers!
> > > -Yogesh
> > >
> >
> >
> >
> > --
> > http://people.apache.org/~britter/
> > http://www.systemoutprintln.de/
> > http://twitter.com/BenediktRitter
> > http://github.com/britter
> >
>

[beanutils2] Checkstyle Errors

[Yogesh Rao]

Hi,

I was planning to contribute to the bu2 project and thought fixing
checkstyle errors should be a good start to it.
I ran the mvn checkstyle:checkstyle command to generate the report which
ended up showing 3218 errors. I wanted to know if all the errors needs to
fixed or only few specific ones.

Please suggest ..

Cheers!
-Yogesh