Jody Garnett ha scritto:
>> Humm... no? The renderer is doing the job of building in the extra
>> filters into the query. Building them just to pass down a hint that
>> only the geometry part should be used is backwards.
>>
>> Read again my mail: the parameter that control how many filters the
>> renderer will add to the original query is already there, I'm
>> just proposing to expose it to the admins.
> 
> I read your mail - and was thinking about the part where what you want
> it to do depends on what the style is trying to accomplish.
> 
> With that in mind - a single setting will help for performance tests
> today; but it sounds like the max number of OR'd rules is something
> that has different results depending on the table - hence the idea of
> a per style (or per table) setting.

Yup, that's what I was discussing in my original mail as long term
approaches:
- have the filter be defined at the layer level (since we already have
   the "definition query" support in the code that is not exposed in the
   GUI)
- have some vendor option in the SLD at the Style, FeatureTypeStyle or
   Rule level to allow specifying whether a filter is actually filtering
- add support for classification functions so that whatever is in the
   rule filter is... actually a filter you want to pass down

> I see that this boils down to the following:
> - rendererParams.put("maxFiltersToSendToDatastore", new Integer(20));
> 
> With that in mind I see how what you describe is a nice patch +1

Cheers
Andrea

-- 
Andrea Aime
OpenGeo - http://opengeo.org
Expert service straight from the developers.

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to