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

Replace UUIDv7 keys #11

Closed sydduckworth closed 1 year ago

sydduckworth commented 1 year ago

We currently use UUIDv7 as time-ordered keys for various files in ella.

The uuid crate recommends v7 for this use case, however since the spec that includes v7 isn't actually stabilized yet, you have to use the uuid_unstable flag to access the v7 feature.

I initially thought this would be fine, but as it turns out it basically makes the crate unusable as a binary and difficult to use as a library, so I think the best course of action is to replace the v7 UUIDs.

It's a bit of a hack, but as a temporary workaround we can just manually re-implement the v7 logic from the uuid crate internally. Then we can switch it out once the feature is stabilized.