Hi all, in an effort to reduce our code base size, and remove un-maintained moduels, I've prepared a couple of pull requests for GeoTools and GeoServer that provide the final step of a migration from gt-geojson to gt-geojsonstore (which was initiated by Ian):
- https://github.com/geotools/geotools/pull/3723 - https://github.com/geoserver/geoserver/pull/5549 As indicated in the pull requests, one of the potential issues I'm seeing, is that gt-geojsonstore bundles in the same package low level conversion between features and GeoJSON, as well as a data store. Since both gs-restconfig and gs-wms depend on some GeoJSON parsing abilities, currently provided by gt-geojson, we'll end up having the GeoJSON data store always included in the GeoServer core package. GeoJSON is ubiquitous, but at the same time, not a well performing data source (text, no indexing). I guess the module could be split into two parts, gt-geojson-core and gt-geojson-store, but I'm not sure if the potential issues are worth the effort (suggestions? volunteers?). Once gt-geojson is not used anymore, it could be removed (maybe in time for GeoTools 27.0?). This would also remove most of the usage points of json-simple, a library that has last been released in 2012... most, but not all. The gt-mbstyle module JSON parsing is built on top of it. Not a big deal, it's a plugin, but pointing that out. The parsing could be replaced by jackson-core + jackson-databind, or to keep things smaller, jackson-core + jackson-jr-stree <https://github.com/FasterXML/jackson-jr> (json-smart is another possibility, still alive, and used in the authkey module... though honestly I'd prefer to reduce the number of JSON libs we use). In any case, I'm not volunteering for this upgrade, simply pointing out the issue. Cheers Andrea == GeoServer Professional Services from the experts! Visit http://bit.ly/gs-services-us for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions Group phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 333 8128928 https://www.geosolutionsgroup.com/ http://twitter.com/geosolutions_it ------------------------------------------------------- Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
