Daniele we may have to move on this, and the jaitools earlier as the release is 
getting held up. 

--
Jody Garnett

On 19/07/2013, at 2:45 AM, Daniele Romagnoli 
<daniele.romagn...@geo-solutions.it> wrote:

> A side note:
> I will be away from tomorrow to monday so I can't check it back before 
> tuesday.
> I have put a note on my agenda to check it again when I'm back.
> 
> Cheers,
> Daniele
> 
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more 
> information.
> ==
> Ing. Daniele Romagnoli
> Senior Software Engineer
> 
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054  Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax:      +39 0584 1660272
> 
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
> 
> -------------------------------------------------------
> 
> 
> On Thu, Jul 18, 2013 at 6:41 PM, Daniele Romagnoli 
> <daniele.romagn...@geo-solutions.it> wrote:
>> Hi Brett,
>> I did a quick look to the patch and it seems good to me.
>> The only thing which I didn't understand is why you allow to compare 
>> DateRange with NumberRange (the latest "else if" returning 1 and the last 
>> "return -1")
>> Ranges to be compared should be of the same type (since an additional domain 
>> is made of homogeneous objects... numbers or dates or ...)
>> 
>> Can you fix that?
>> Cheers,
>> Daniele
>> 
>> 
>> ==
>> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more 
>> information.
>> ==
>> Ing. Daniele Romagnoli
>> Senior Software Engineer
>> 
>> GeoSolutions S.A.S.
>> Via Poggio alle Viti 1187
>> 55054  Massarosa (LU)
>> Italy
>> phone: +39 0584 962313
>> fax:      +39 0584 1660272
>> 
>> http://www.geo-solutions.it
>> http://twitter.com/geosolutions_it
>> 
>> -------------------------------------------------------
>> 
>> 
>> On Thu, Jul 18, 2013 at 4:29 PM, Brett Walker <brett.wal...@geometryit.com> 
>> wrote:
>>> Hi
>>> 
>>> Could someone review PR #237.
>>> 
>>> I found and fixed a build error when using JDK7. It is in unsupported 
>>> module but it still broke the build.
>>> 
>>> Additional information is found in the commit message of the pull request.
>>> 
>>> Thanks,
>>> 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
>>> 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
------------------------------------------------------------------------------
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

Reply via email to