Re: [Geotools-devel] JSONPointer function backport

2019-03-07 Thread Andrea Aime
Backport PRs are up: https://github.com/geotools/geotools/pull/2298 https://github.com/geotools/geotools/pull/2299 Cheers Andrea On Mon, Feb 11, 2019 at 10:10 AM Andrea Aime wrote: > Hi, > I've merged the PR adding the JSONPointer function. > How do others feel about a backport? It's a pure

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Jody Garnett
Fair enough. That said we are still on for release next week. Everything is clear on the geotools side of things (working on documentation as a nice to have). -- Jody Garnett On Mon, 11 Feb 2019 at 07:48, Andrea Aime wrote: > Thank you, but no thank you! You know how I feel about last minute

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Andrea Aime
Thank you, but no thank you! You know how I feel about last minute changes, been burned too many times ;-) Cheers Andrea On Mon, Feb 11, 2019 at 4:46 PM Jody Garnett wrote: > I read you correctly, I think this function is useful and would make a > good addition to 21.0. It will of course also

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Jody Garnett
I read you correctly, I think this function is useful and would make a good addition to 21.0. It will of course also make a good addition to 21.1 in a couple months time... -- Jody Garnett On Mon, 11 Feb 2019 at 07:31, Andrea Aime wrote: > Hi Jody, > I believe you misread my message, I'm

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Andrea Aime
Hi Jody, I believe you misread my message, I'm explicitly saying to wait until GeoTools 21.0 is out, and only then do the backport. I'm asking regardless because I would like to avoid waiting one month (which, if done, would bring me very close to the 20.3 release date, and I'd rather avoid it)

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Jody Garnett
We usually do a call out for any last moment fixes and back ports ahead of a release ... if this is you answering the call I am fine with the back port. The new dependency is the only thing giving me pause. -- Jody Garnett On Mon, 11 Feb 2019 at 01:11, Andrea Aime wrote: > Hi, > I've merged

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Ian Turton
+1 Ian On Mon, 11 Feb 2019 at 09:11, Andrea Aime wrote: > Hi, > I've merged the PR adding the JSONPointer function. > How do others feel about a backport? It's a pure addition, should not > affect anything else, but of course > adds a new dependency to main. > > My proposal: I'd wait for 21.0

Re: [Geotools-devel] JSONPointer function backport

2019-02-11 Thread Simone Giannecchini
+0 Regards, Simone Giannecchini == GeoServer Professional Services from the experts! Visit http://bit.ly/gs-services for more information. == Ing. Simone Giannecchini @simogeo Founder/Director GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: