paleolimbot opened a new issue, #1033:
URL: https://github.com/apache/sedona-db/issues/1033

   In general, we use `DataFusionError` everywhere in the sedona crates, except 
for the few crates that don't involve DataFusion (e.g., sedona-geometry, 
sedona-raster). Sedona raster rather awkwardly uses ArrowError, because that is 
the error type it more directly uses most of the time. It's fairly common for 
sedona-geometry's error to be used in kernels, where there is a lot of 
`.map_err()`s instead of `?`s.
   
   We can consolidate errors to a crate `sedona-error` that has features 
`arrow` and `datafusion`, which, when enabled, add error variants and `From<>` 
implementations for ArrowError and DataFusionError. This will hopefully make 
the error handling between crates a bit more unified and reduce the number of 
dedicated error classes we need.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to