[ 
https://issues.apache.org/jira/browse/SEDONA-272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17707908#comment-17707908
 ] 

Jia Yu commented on SEDONA-272:
-------------------------------

Given that Sedona can utilize Spark's built-in JSON data source to read 
GeoJSON, I feel we can gradually deprecate our own GeoJSON reader. Instead, we 
can implement this logic in ST_GeomFromGEOJSON. What do you think?

 

Please check the tutorial here to learn how to use ST_GeomFromGeoJSON + Spark 
built-in JSON data source: 
[https://sedona.apache.org/latest-snapshot/tutorial/sql/#load-geojson-using-spark-json-data-source]

> GeoJsonReader.readToGeometryRDD should handle GeoJSON documents missing 
> geometry field
> --------------------------------------------------------------------------------------
>
>                 Key: SEDONA-272
>                 URL: https://issues.apache.org/jira/browse/SEDONA-272
>             Project: Apache Sedona
>          Issue Type: Improvement
>            Reporter: Tomasz Taraś
>            Priority: Minor
>
> Currently when document is missing geometry field NullPointerException is 
> raised. While the GeoJSON spec says that document must have geometry key but 
> allows geometry field to contain JSON null value I think we should handle 
> both cases gracefully and read those rows as null geometry.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to