Patches look good. +1.

On Sun, Jan 30, 2011 at 1:16 AM, Andrea Aime <andrea.aime@geo-solutions.it> wrote:
On Sun, Jan 30, 2011 at 12:56 AM, Jody Garnett <jody.garnett@gmail.com> wrote:
> Sounds good; did the change have the expected performance gain?

I could not try that out, but for sure it would have had.

The thing is, we found the issue in a customer project that is GT2 based, and
to solve it quickly we suggested them to replace the filter factory with a  new
one generating a literal that would not do all the amount of guessing the old
one was doing, so there the problem is already solved

If it's ok I'm going to commit the changes on trunk (now that 2.7.x has
been branched off) and see if something else breaks

Cheers
Andrea

--
Ing. Andrea Aime
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy

phone: +39 0584962313
fax:     +39 0584962313

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf

-----------------------------------------------------

------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires
February 28th, so secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel



--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.