[Tarantool-patches] [PATCH] vclock: rename vclock_reset to vclock_set
Konstantin Osipov
kostja.osipov at gmail.com
Wed Apr 15 19:09:02 MSK 2020
* Cyrill Gorcunov <gorcunov at gmail.com> [20/04/15 17:16]:
> The `reset` keyword is too overloaded, its traditional
> context is to setup some value either to zero or to default.
>
> That is not what we're doing in this helper: we rather set up
> a new value to the replica_id in the map, keeping consistency
> of the sum.
> Thus lets name it as it should -- vclock_set.
Well, one could argue that it's impossible to set a vclock
component without violating consistency - vclock should never go
back.
reset() indicates that it can unwind the clock.
--
Konstantin Osipov, Moscow, Russia
More information about the Tarantool-patches
mailing list