I have been catching up with the GeoNode project this week (interestingly
the project is using GDAL 2.0 which we were discussion earlier in the week).

Right now the project is running a fork of geotools to have the following
fix (from #997 <https://github.com/geotools/geotools/pull/997>):

         Date converted = null;
         if (target == PostGISDialect.XDate.class) {
             converted = Converters.convert(expression.evaluate(null),
Date.class);
         }
         if (converted != null) {
             return converted.getTime();
        } else {
             return super.evaluateLiteral(expression, target);
        }

I rejected this fix, since it violated the API contract (by returning a
Date rather than the requested XDate). The quickest thing I can think of is
to make XDate a real class and write a converter that handles it
appropriately.

This means that the code would return XDate instances (XDate does extend
Date so I would hope client could would remain happy?).
--
Jody Garnett
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial! http://pubads.g.doubleclick.net/
gampad/clk?id=1444514301&iu=/ca-pub-7940484522588532
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to