CSVDataStore is used as a tutorial in GeoTools (as was PropertyDataStore
before it).

I think it could use a QA/Readability review for the different CSV reading
strategies, I found the code to be amazing convoluted to follow (with some
API choices made so test cases could be written against strings).

--
Jody Garnett

On 2 June 2015 at 12:13, Torben Barsballe <[email protected]>
wrote:

>
> One warning - Last time I interacted with the CSV datastore (in March) and
> GeoServer importer, it looked like it was still missing a few things. It
> seems the CSV datastore does not handle geometry columns especially well.
> CSV Import via the GeoServer importer UI only supports CSVs as lists of
> attributes (no geometry). Via the Importer REST API you can specify Lat/Lon
> columns. Support for WKT columns exists, but there is no way of actually
> importing/specifiying columns as WKT as far as I can see, although this
> might also be doable using the REST API.
>
> A custom config panel seems like a very good idea, and should include some
> way to specify the Geometry column(s) (Lat/Lon or WKT).
>
> Given that Property datastores
> <http://docs.geoserver.org/latest/en/user/data/app-schema/data-stores.html#property-file>
> are supported in core GeoServer, it would make just as much sense to also
> include the CSV datastore in core GeoServer.
>
> Torben
>
> On Tue, Jun 2, 2015 at 12:21 AM, Andrea Aime <[email protected]
> > wrote:
>
>> On Tue, Jun 2, 2015 at 9:15 AM, Andrea Aime <[email protected]
>> > wrote:
>>
>>> I think it would be a good win to make a config panel for it
>>>
>>
>> As in, a custom one, since some params are making sense only for some
>> choice of mapping
>> strategy
>>
>>
>>
>> --
>> ==
>> GeoServer Professional Services from the experts! Visit
>> http://goo.gl/it488V for more information.
>> ==
>>
>> Ing. Andrea Aime
>> @geowolf
>> Technical Lead
>>
>> GeoSolutions S.A.S.
>> Via Poggio alle Viti 1187
>> 55054  Massarosa (LU)
>> Italy
>> phone: +39 0584 962313
>> fax: +39 0584 1660272
>> mob: +39  339 8844549
>>
>> http://www.geo-solutions.it
>> http://twitter.com/geosolutions_it
>>
>> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>>
>> Le informazioni contenute in questo messaggio di posta elettronica e/o
>> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
>> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
>> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
>> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
>> darcene notizia via e-mail e di procedere alla distruzione del messaggio
>> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
>> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
>> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
>> principi dettati dal D.Lgs. 196/2003.
>>
>>
>>
>> The information in this message and/or attachments, is intended solely
>> for the attention and use of the named addressee(s) and may be confidential
>> or proprietary in nature or covered by the provisions of privacy act
>> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
>> Code).Any use not in accord with its purpose, any disclosure, reproduction,
>> copying, distribution, or either dissemination, either whole or partial, is
>> strictly forbidden except previous formal approval of the named
>> addressee(s). If you are not the intended recipient, please contact
>> immediately the sender by telephone, fax or e-mail and delete the
>> information in this message that has been received in error. The sender
>> does not give any warranty or accept liability as the content, accuracy or
>> completeness of sent messages and accepts no responsibility  for changes
>> made after they were sent or for other risks which arise as a result of
>> e-mail transmission, viruses, etc.
>>
>> -------------------------------------------------------
>>
>>
>> ------------------------------------------------------------------------------
>>
>> _______________________________________________
>> Geoserver-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>
>>
>
>
> ------------------------------------------------------------------------------
>
> _______________________________________________
> Geoserver-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
>
------------------------------------------------------------------------------
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to