convert by-user-id index to messagepack #3995
Conversation
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
There was a problem hiding this comment.
How about we force-refresh the index during startup? I'm just a little worried that spaces end up missing from the index when something goes wrong while updating the index. It's kinda self-healing when the according index is touched again, but esp. for the user cache this could take a while or not happen at all.
aduffeck
force-pushed
the
msgpack-index
branch
2 times, most recently
from
32e1b91 to
2babdce
Compare
aduffeck
requested review from
labkode,
ishank011,
glpatcern and
a team
as code owners
|
@micbar yes, with the current implementation the first instance running the new code will start the migration which removes the old indexes. Older instances would thus not be able to list spaces anymore. Is that a problem? We could make it populate both indexes at first for a smoother upgrade, but I'm not sure the added complexity is justified. |
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
No description provided.