Were you expecting this in the current release? If so please advise.
(And thanks for stepping up with all the pull request Brett)
--
Jody Garnett
On Thursday, 18 July 2013 at 2:58 AM, Simone Giannecchini wrote:
> Ciao Bret,
> I had a quick look and I am happy withe changes. I will create a jira
> and backport to 9.x
>
> Regards,
> Simone Giannecchini
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for
> more information.
> ==
>
> Ing. Simone Giannecchini
> @simogeo
> Founder/Director
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
> mob: +39 333 8128928
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
>
> On Wed, Jul 17, 2013 at 3:28 AM, Brett Walker
> <brett.wal...@geometryit.com (mailto:brett.wal...@geometryit.com)> wrote:
> >
> >
> > -----Original Message-----
> > From: Michael Bedward [mailto:michael.bedw...@gmail.com]
> > Sent: Wednesday, 17 July 2013 11:18 AM
> > To: Brett Walker
> > Cc: geotools-devel@lists.sourceforge.net
> > (mailto:geotools-devel@lists.sourceforge.net); jody.garn...@gmail.com
> > (mailto:jody.garn...@gmail.com)
> > Subject: Re: Pull Request to upgrade JAITools
> >
> > Hi Brett,
> >
> > Thanks for this work.
> >
> > Yes, there were substantial changes to RangeLookupTable and related classes
> > to fix thread safety, so it's no surprise that you had to hack things at
> > the GeoTools end. However, I don't know the CoverageUtilities code at all -
> > it would be best to ask Simone, Andrea or Daniele to review it.
> >
> > Michael
> >
> >
> > On 16 July 2013 22:21, Brett Walker <brett.wal...@geometryit.com
> > (mailto:brett.wal...@geometryit.com)> wrote:
> > > Hi,
> > >
> > > I have made a pull request to upgrade JAITools to version 1.3.1.
> > >
> > > To get the build to pass it seems that I had to brutalise a method in
> > > CoverageUtilities.java
> > > (https://github.com/brett-walker/geotools/commit/35ee8f8434f2dbb435622
> > > 671f725a08e295df848)
> > >
> > > A question for Jody or Michael.
> > >
> > > Do the changes look right? Have I changed the method's meaning too
> > > drastically? Or is the change from version 1.3.0 to 1.3.1 to drastic
> > > for CoverageUtilities do be altered that the upgrade is not possible?
> > >
> > > The process-raster module, of which the CoverageUtilities is apart, is
> > > in the unsupported part of GeoTools.
> > >
> > > Brett
>
> ------------------------------------------------------------------------------
> See everything from the browser to the database with AppDynamics
> Get end-to-end visibility with application monitoring from AppDynamics
> Isolate bottlenecks and diagnose root cause in seconds.
> Start your free trial of AppDynamics Pro today!
> http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> (mailto:GeoTools-Devel@lists.sourceforge.net)
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
