georust / geozero

Zero-Copy reading and writing of geospatial data.
Apache License 2.0
321 stars 30 forks source link

Handle nested properties in GeoJSON reader and writer #208

Closed kylebarron closed 1 month ago

kylebarron commented 2 months ago

The GeoJSONReader currently silently omits any nested property. This serializes the Value for an Array or Object back to a string to store within a ColumnValue::Json.

The GeoJSONWriter currently silently omits any ColumnValue::Json or ColumnValue::Binary. This PR writes the contained JSON string to the output writer. It leaves writing binary to a future PR (which should probably at least have the option of converting to hex?)

I'm not sure why but I was unable to run tests locally because they failed with

  = note: ld: library 'pq' not found

I'm guessing this has something to do with Postgres' libpq, but brew install libpq says it's already installed.

kylebarron commented 1 month ago

Anything else I can do to push this along?