Thanks for explanation! LGTM. Sergos > On 12 Oct 2021, at 16:38, Serge Petrenko wrote: > > > > 12.10.2021 12:56, sergos пишет: >> Hi! >> >> This one looks unnecessary, since you rework it in second part. >> >> Sergos. > Hi! Thanks for the review! > > The rework you mention is simply a refactoring, while this commit introduces > a change in behaviour. That's why I extracted it. >>> On 5 Oct 2021, at 16:18, Serge Petrenko wrote: >>> >>> Once an anonymous replica tries to register, it reconnects to every >>> other instance in its replication list in order to receive updated >>> ballots and choose someone to register on. >>> >>> Make the instance wait until it connects to quorum before letting it >>> choose the node to register on. >>> --- >>> src/box/box.cc | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/src/box/box.cc b/src/box/box.cc >>> index 0b12b1328..219ffa38d 100644 >>> --- a/src/box/box.cc >>> +++ b/src/box/box.cc >>> @@ -1404,7 +1404,7 @@ box_set_replication_anon(void) >>> * them can register and others resend a >>> * non-anonymous subscribe. >>> */ >>> - box_sync_replication(false); >>> + box_sync_replication(true); >>> /* >>> * Wait until the master has registered this >>> * instance. >>> -- >>> 2.30.1 (Apple Git-130) >>> > > -- > Serge Petrenko