matrix-org / rust-synapse-compress-state

A tool to compress some state in a Synapse instance's database
https://pypi.org/project/synapse-auto-compressor/
Apache License 2.0
142 stars 32 forks source link

Fix save ratio check #90

Open Fizzadar opened 2 years ago

Fizzadar commented 2 years ago

Currently the compressor will rewrite state even if the result consists of the same number of rows which doesn't match the log statement.

erikjohnston commented 2 years ago

The clippy lint will be fixed by #92 #91

The test failure is annoying, as it looks like the test cases that we're using ends up generating the same number of rows?

clokep commented 2 years ago

I think this is waiting on @Fizzadar, so I'm going to remove this from the review queue.