Yep I know this is "one of these cases" - this is partly why I am so 
enthusiastic to see it being addressed :-)

The nature of this change (effecting other modules) does put it in proposal 
territory; and we really need to get feedback from effected modules 
(specifically gt-render). I am available on IRC if you want to chat through 
these issues.

As indicated I am enthusiastic for this change and think it is a good idea; we 
just need to sort out the consequences.

If you read aaime's email he was going through some of the expectations for 
xpath evaulation. I don't have a sample xml file and xpath tool to answer those 
questions - but tin my very first email response to this subject I indicated 
that I would like to see us match the xpath experience (so that the Filter and 
SLD data structures we produce are indeed handled correctly). 

-- 
Jody Garnett

On Wednesday, 11 May 2011 at 11:35 AM, Ben Caradoc-Davies wrote: 
> Jody,
> 
> this change is precisely one of these cases, where the existing 
> app-schema implementation has ugly workarounds for filtering on 
> multivalued properties. Niels is working on performance improvements in 
> the way SQL queries are generated by app-schema, and rather than 
> perpetuating our existing workarounds, he is using this opportunity to 
> improve the core GeoTools API and implementation. If you recall his 
> previous work on filters (to support app-schema WMS), you'll remember 
> that he also took this approach in that work. This is an example of the 
> kind of improvement you are advocating.
> 
> Kind regards,
> Ben.
> 
> On 10/05/11 18:09, Jody Garnett wrote:
> > The work in app-schema has opened up a number of design issues (handling of 
> > multiple values is one that was left on the table when this project 
> > started; ben turned up a need for FeatureCollection to have a descriptor 
> > for feature members; need some feature builders created etc...) and I am 
> > worried that they are not getting addressed and are going to be left for a 
> > rainy day / code sprint.
> > I am not asking you to address all of these today; just the one you are 
> > exposing the code base to.
> 
> -- 
> Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au>
> Software Engineering Team Leader
> CSIRO Earth Science and Resource Engineering
> Australian Resources Research Centre
> 
------------------------------------------------------------------------------
Achieve unprecedented app performance and reliability
What every C/C++ and Fortran developer should know.
Learn how Intel has extended the reach of its next-generation tools
to help boost performance applications - inlcuding clusters.
http://p.sf.net/sfu/intel-dev2devmay
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to