Georgy Moshkin authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

Originally it was meant to be a key for distinguishing outdated peer
info. But this was never actually fully implemented. Now that we have
the governor and it's synchronizing mechanism there's even less point
to this field. And now that this legacy code impedes refactoring, we
remove it at least until we have a better idea of how to implement it.

Yaroslav Dynnikov authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

Eliminate code duplication, use `define_str_enum` macro from tarantool-module.
Remove personalized `FromStr::Err` for every enum defined that way.

Cases covered:
- when someone goes offline
- whenever vshard gets reconfigured

Georgy Moshkin authored 2 years ago and Georgy Moshkin committed 2 years ago

Currently stores:
- replicset_id
- replicaset_uuid
- sharding weight
- replication master id

Every field of Peer now has a corresponding unit struct defined in the
`peer_field` module, which implements the `PeerFieldDef` trait which in
turn contains information about the field's name, tarantool type and
rust type.

This information currently is used in the `field_by_raft_id` method
and is generated by the `define_peer_fields` macro, which also
automates the generation of the format (`peer_format` function) for the
"raft_group" space.