On Mon, Jun 6, 2011 at 2:39 AM, Jody Garnett <jody.garn...@gmail.com> wrote:

>
> --
> Jody Garnett
>
> On Monday, 6 June 2011 at 2:04 PM, Justin Deoliveira wrote:
>
> Ok, i bit the bullet on this one and just went and updated all the
> FilterVisitor implementations as best I could. For some I was able to come
> up with implementations based on the nature of the class... and for others
> for which I don't know I just made all the methods throw
> UnsupportedOperationException.
>
> Wow justin you rock :-)
>
> I would like some way to indicate to module maintainers when a code review
> is needed (as with your visitors that throw unsupported operation
> exception). Could we create a subtask of GEOT-2367 for "gt-cql temporal cql
> support" and indicate in the description that FilterToCQL needs review.
>

Ok, created subtasks and assigned them accordingly. Do you need a substask
for opengis and main Jody?

>  The current patch can be found here:
>
>   http://jira.codehaus.org/browse/GEOT-2367
>
> Now about gt-temporal Jody has requested that we roll the module into main
> and api/opengis. I think I agree... one less module would be nice. I have
> not heard back from anyone who has their name on hat module yet so unless
> someone objects I will probably try to do that this week.
>
> Oh and also btw, while it is not present in the patch I have also
> implemented all the XML bindings for the new temporal classes... The reason
> they are not on the patch is because they are mixed in and require a bunch
> of changes that are on the current wfs2 branch. So breaking them out into a
> clean patch will take some work... will do that this week.
>
> That will be a busy week; thanks for you hard work on this one Justin.
>
> Jody
>
>


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Discover what all the cheering's about.
Get your free trial download today. 
http://p.sf.net/sfu/quest-dev2dev2 
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to