On Wed, Apr 18, 2012 at 11:09 PM, Jody Garnett <[email protected]>wrote:
> Just a heads up.
>
> There is a nice small patch here (
> https://jira.codehaus.org/browse/GEOT-4110) that changes how ECQL
> *output* behaves.
>
> Previous:
>
> ECQL.toCQL( Filter.INCLUDE ) ---> "1=1"
> ECQL.toCQL( Filter.EXCLUDE ) ---> "1=0"
>
> After the patch:
>
> ECQL.toCQL( Filter.INCLUDE ) ---> "INCLUDE"
> ECQL.toCQL( Filter.EXCLUDE ) ---> "EXCLUDE"
>
> The ECQL parser is already able to handle parsing these tokens.
>
>
> Now here is the interesting part - what will the effects of this change be
> on downstream code?
>
> My understanding is that GeoServer spends a lot of time Parsing ECQL - so
> that should not be effected.
>
> Is there any other consequence to this change that can be imagined?
>
Hmm... as long as CQL/ECQL can parse whatever is produced by the toCQL
method I don't see a problem.
Wondering if anything wrong might happen if the filters are combined with
something else, e.g.,
INCLUDE AND ATT = 'ABC'
(I know the above does not make sense but there is quite a bit of filter
manipulation code that
ends up with situation like the above without calling the simplifying
filter visitor later)
Cheers
Andrea
--
-------------------------------------------------------
Ing. Andrea Aime
GeoSolutions S.A.S.
Tech lead
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 962313
mob: +39 339 8844549
http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.youtube.com/user/GeoSolutionsIT
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf
-------------------------------------------------------
------------------------------------------------------------------------------
For Developers, A Lot Can Happen In A Second.
Boundary is the first to Know...and Tell You.
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
