[Tarantool-patches] [RFC 2/4] qsync: move synchro quorum update to separate routine
gorcunov at gmail.com
Fri Nov 20 14:01:48 MSK 2020
On Fri, Nov 20, 2020 at 01:06:55PM +0300, Serge Petrenko wrote:
> IMO it'd be better to name the new function somewhat similar to
> `box_update_replication_synchro_quorum` and leave it in box.cc
> This way you avoid adding 2 new dependencies to replication.cc
> and leave all the code dealing with reconfiguration at one place.
> Besides, replication.cc already depends on box, so it won't be a problem
> to call update_replication_synchro_quorum there.
Yeah, I don't mind, thanks!
> Load_cfg.lua will say something similar, when replication_synchro_quorum is
> a number:
> log.info("set '%s' configuration option to %s", key,
> So this say_info belongs to the trigger you invoke on replica
It will say "set ... to n+2/1" while we're printing the evaluated
integer value which is more informative.
More information about the Tarantool-patches