[tarantool-patches] Re: [PATCH v2 2/3] box: don't set cluster uuid and instance id in initial snapshot

[Konstantin Osipov]

* Vladimir Davydov <vdavydov.dev at gmail.com> [19/02/15 15:26]:
> They aren't needed there as we reset them anyway once the snapshot is
> replayed on initial bootstrap. OTOH having them results in calling
> replica_{set,clear}_id twice on initial bootstrap, which will look weird
> when I patch them to log the ids. So let's remove them from the initial
> snapshot. This makes the initial bootstrap impossible to recover from as
> it is, but that shouldn't be an issue since one can always bootstrap a
> new instance in a normal way. This also allows us to make cluster uuid
> truly immutable (currently, one can update it with REPLACE).

This patch is Ok to push.


-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov