Hi Andrea,

In the JIRA issue is a stack trace that caused the issue.

It is my understanding that for a TreeSet that the elements are supposed to 
implement Comparable or you need to supply a Comparator.

DateRange and NumericRange don't implement Comparable.

I could be wrong here.

Brett

Sent from my iPad

On 20/07/2013, at 11:10 PM, "Andrea Aime" 
<andrea.a...@geo-solutions.it<mailto:andrea.a...@geo-solutions.it>> wrote:

On Sat, Jul 20, 2013 at 2:58 PM, Andrea Aime 
<andrea.a...@geo-solutions.it<mailto:andrea.a...@geo-solutions.it>> wrote:
Generally speaking, the comparator for those two treeset of Object there should 
take care first of the case Comparable
against Comparable, and then try to do comparisons of date and number ranges as 
a fallback.
If the contents of the set are mixed type is most certainly an error, that 
should throw an exception.

Anyways, this is my opinion after a cursory look, let me give it a spin with 
Java 7 and see

>From what I can see the code is always building range sets because the 
>underlying NetCDF lib data model
has a min and max for each and every variable, I guess there is no way to tell 
if a variable is a flat scalar?

Unfortunately my understanding of NetCDF is limited, I'd prefer to have a talk 
with Daniele next week when
he comes back about it before moving on.
One thing is sure, unless I'm missing something, the code should not allow a 
mix of numeric and date ranges,
but should throw an exception instead

Soo... please give us a few more days to figure it out. JDK 7 is unsupported 
anyways, it's not a big deal if we have
the build broken for a little while.

Cheers
Andrea

--
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more 
information.
==

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39  339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------
------------------------------------------------------------------------------
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