CerebusOSS / ella

A streaming time-series datastore for low-latency applications
https://docs.rs/ella/
Apache License 2.0
2 stars 0 forks source link

Style: Better error handling #18

Open sydduckworth opened 1 year ago

sydduckworth commented 1 year ago

Right now we're using thiserror for error handling to provide a single ella::Error type across all crates. However, the current implementation is a bit of a mess, and it can be somewhat difficult to track down an error because the error messages are often generic (e.g. "datafusion error"). A potentially better approach would be to use snafu instead, which encourages more specific error messages.