[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