On 17 December 2014 at 12:48, Travis Brundage <tbrund...@boundlessgeo.com>
wrote:
>
> Hey, Travis here again. Taking a look into the updated CSVDataStore in
> geoserver, my general impression is that its design was updated to be more
> general and extensible using strategy objects. A few new capabilities
> plugged in with it. Otherwise functionality is the same, just an
> architecture change. I'll be backporting it to geotools.
>
>
So far ran into a few concerns. The CSVDataStore tests in geoserver rely on
> some importer utility classes to load a .zip file. To me it seemed like a
> better idea to just modify the test to use geotools' test data instead, but
> let me know if I should do something else.
>
Sounds good.
Should I append the geotools license to the existing one coming from
> geoserver? You can see the code so far here.
> <https://github.com/travislbrundage/geotools/commit/309468689c68fb80bcebf0a5c27b02d142f990dd>
>
In general we "ask" with an email on geoserver-devel, wait for a PMC member
to say +1 and then keep going (we have to be this formal because we are
changing from GPL to LGPL). GeoSever and GeoTools are well practiced at
working together, sometimes when we ask another project they end up saying
"no".
Currently the gt-csv module is under unsupported. Where should it be in the
> future? (extension? plugin?)
>
The developers guide has the details on how unsupported and plugin work.
For now it stays in unsupported until we can meet the QA and documentation
requirement. But once again see the developers guide for details (we may
have to clean up a few pages as you have questions).
With updating and moving the gt-csv module, will I also need to make some
> update for maven to build it properly? (currently builds fine, leaving it
> in unsupported)
>
We may be in position to ask the module be accepted as a plugin after it is
updated. For now lets update and remove the fork from GeoServer.
Tutorial update is still to come.
>
Yep, that will be a bit of work. And a great chance to see if the
additional functionality makes sense.
--
Jody
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
