-- 
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.
> 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 

------------------------------------------------------------------------------
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