Re: JsonOutput serialization options feature

2016-09-23 Thread Graeme Rocher
Great, yeah that would be a big help, since efficiency over the wire for JSON services is a big deal and the unicode escaping doesn't help there. Also, can we consider https://github.com/apache/groovy/pull/429 as an improvement? Again it makes using StreamingJsonBuilder more efficient and

Re: JsonOutput serialization options feature

2016-09-22 Thread John Wagenleitner
On Wed, Sep 21, 2016 at 3:22 AM, Graeme Rocher wrote: > Hi John, > > Looks very useful, so +1 > > One thing that would also be useful is to be able to configure the > ability to disable unicode encoding. It doesn't seem to be recommended > for all circumstances. See: > >

Re: JsonOutput serialization options feature

2016-09-11 Thread John Wagenleitner
On Sun, Sep 11, 2016 at 11:51 AM, Kostas Saidis wrote: > +1, definitely! > > If it is possible, the dateFormat method should also accept an optional > Locale parameter (as mentioned in TODO comment in the source), with a > default value of Locale.US. > > Cheers, > Kostas > >

Re: JsonOutput serialization options feature

2016-09-11 Thread Kostas Saidis
+1, definitely! If it is possible, the dateFormat method should also accept an optional Locale parameter (as mentioned in TODO comment in the source), with a default value of Locale.US. Cheers, Kostas On 11/9/2016 6:59 πμ, John Wagenleitner wrote: Groovy Json users, I recently worked on a

Re: JsonOutput serialization options feature

2016-09-11 Thread Robert Stagner
+1 from me. I like the ability to pick and choose what I can exclude On Sun, Sep 11, 2016 at 8:47 AM Guillaume Laforge wrote: > More options sounds like a good idea. > > On Sun, Sep 11, 2016 at 5:43 PM, Suderman Keith wrote: > >> +1 >> >> I haven't had a

Re: JsonOutput serialization options feature

2016-09-11 Thread Guillaume Laforge
More options sounds like a good idea. On Sun, Sep 11, 2016 at 5:43 PM, Suderman Keith wrote: > +1 > > I haven't had a chance to look at your pull request, but the lack of > configurability is the main reason I use Jackson instead of the Groovy > JsonBuilder/JsonOutput classes.

Re: JsonOutput serialization options feature

2016-09-11 Thread Suderman Keith
+1 I haven't had a chance to look at your pull request, but the lack of configurability is the main reason I use Jackson instead of the Groovy JsonBuilder/JsonOutput classes. Keith > On Sep 10, 2016, at 8:59 PM, John Wagenleitner

JsonOutput serialization options feature

2016-09-10 Thread John Wagenleitner
Groovy Json users, I recently worked on a project where it would have been helpful if JsonOutput/JsonBuilder/StreamingJsonBuilder had options for controlling the output. I noticed there were some open tickets [1] for feature requests around this so decided to try to add some options to control