Contribute three LRS WPS processes

Michael Bedward added a comment - 02/Apr/12 5:44 PM Andrea, Changed the component from WPS to Process and provisionally assigned this one to you since it seems to be putting things into the process.gs package. Michael

Andrea Aime added a comment - 04/Apr/12 3:35 AM First off, thanks for the contribution, LRS processes are certainly welcomed. There are a few issues that will prevent the patch from being committed as is: * it does not respect the geotools coding convetions (evident in the usage of tabs instead of spaces, but this I can fix) * when a feature has a geometry outside of the expected date range we generate a feature with a null geometry? Or an exception? I guess filtering out the feature from the results might be a more desirable behavior? * the processes are memory bound (ok, this one is an efficiency problem, does not prevent commit per se, but I would not be eager of exposing the processes as is on a publicly accessible WPS) * there are no tests checking the proper operation of the processes, someone has to make them before commit can go in An improved patch that I just need to review will fast track the commit, otherwise we'll have to wait for someone with enough time to add at lest the tests and eventual fixes in behavior when the geometry is fully outside of the requested interval (formatting can be fixed quickly while committing the processes).

David R Robison added a comment - 04/Apr/12 10:40 AM This patch includes JUnit tests

David R Robison added a comment - 04/Apr/12 11:21 AM Added a small edit to set point SRID

Andrea Aime added a comment - 15/Apr/12 7:07 PM Patch applied, formatting fixed, the other issues (memory boundness) are for the moment unresolved. If you feel like giving it a shot today I've rolled a SimpleProcessingCollection abstract class that can be used to write streaming processes without having to pull too many hairs