NLnetLabs / krill

RPKI Certificate Authority and Publication Server written in Rust
https://nlnetlabs.nl/projects/routing/krill/
Mozilla Public License 2.0
289 stars 40 forks source link

Let AggregateStore use 'ChangeSets' rather than separate Commands and Events #1078

Closed timbru closed 1 year ago

timbru commented 1 year ago

Currently we save a Command key, and then 0 (in case of an error) or more Events (in case there was a change).

We can simplify this by saving a single ChangeSet structure that includes all this. Just like WalStore does.

This will simplify the logic around finding these files. Each change will be in a single atomic stored key. It will also reduce the number of keys (files for most deployments).

timbru commented 1 year ago

We will also need to re-work command history when we do this: