Re: [Tarantool-patches] [PATCH 0/3] box.execute() and box.cfg() idempotence and locking

[Alexander Turenko <alexander.turenko@tarantool.org>]

On Mon, Jun 22, 2020 at 01:11:12PM +0300, Kirill Yukhin wrote:
> Hello,
> 
> On 13 май 01:18, Alexander Turenko wrote:
> > This patchset provides several fixes for box.execute() and box.cfg()
> > functions when they are called under various circumstances: when a
> > function is saved before box loading and called after it, when a
> > function is called during box loading.
> > 
> > Existence of this patchset does not mean that we'll not implement
> > https://github.com/tarantool/tarantool/issues/4726: I don't know whether
> > we will do or will not, but I intend to fix bugs in the existing code.
> > 
> > https://github.com/tarantool/tarantool/issues/4231
> > Totktonada/gh-4231-box-execute-idempotence
> > 
> > My review is not more sufficient, because I became co-author of the
> > patchset. Igor, can you, please, review it and pass to a second reviewer
> > (I suggest Vlad)?
> > 
> > Alexander Turenko (1):
> >   box: always wait box loading in box.execute()
> > 
> > Maria (2):
> >   box: check whether box is loaded in box.execute()
> >   box: always reconfigure box at non-first box.cfg()
> 
> I've checked your patchset into 2.3, 2.4 and master.

Pushed 'box: always reconfigure box at non-first box.cfg()' to 1.10.

Borrowed <box_is_configured> variable and related comments from 'box:
check whether box is loaded in box.execute()' to correctly apply the
change.

Added an entry to the future 1.10.7 release notes.

Note: Also fixed gh-4221 with gh-4231 in other release notes, I mistyped
it in [1]. Sorry.

[1]: https://lists.tarantool.org/pipermail/tarantool-patches/2020-June/017543.html

WBR, Alexander Turenko.