

<HTML><BODY><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Четверг, 27 февраля 2020, 2:54 +03:00 от Vladislav Shpilevoy <v.shpilevoy@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15827612921252979183_BODY">Hi! Thanks for the patch!<br><br>Please, add a @ChangeLog record.</div></div></div></div></blockquote><div>Hi!</div><div>What kind of a changelog should I write?</div><div>This is a bug fix with no visible behaviour change (speaking from the users point of view)</div><div>--<br>Serge Petrenko</div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><div class="js-helper js-readmsg-msg"><div><div><br>On 26/02/2020 11:00, sergepetrenko wrote:<div class="mail-quote-collapse">> <a href="https://github.com/tarantool/tarantool/issues/4739" target="_blank">https://github.com/tarantool/tarantool/issues/4739</a><br>> <a href="https://github.com/tarantool/tarantool/tree/sp/gh-4739-vclock-assert-v4" target="_blank">https://github.com/tarantool/tarantool/tree/sp/gh-4739-vclock-assert-v4</a><br>><br>> Changes in v4:<br>> - move row skipping logic from recovery to relay<br>> - encode a list of instances whose rows to skip<br>> in SUBSCRIBE request insead of encoding<br>> is_orhpan status<br>><br>> Changes in v3:<br>> - review fixes as per review from Vlad<br>> - instead of skipping rows on replica side,<br>> do it on master side, by patching recovery<br>> to silently follow rows coming from a certain<br>> instance.<br>><br>> Changes in v2:<br>> - review fixes as per review from Kostja<br>><br>> Serge Petrenko (4):<br>> box: expose box_is_orphan method<br>> wal: warn when trying to write a record with a broken lsn<br>> replication: implement an instance id filter for relay<br>> replication: do not relay rows coming from a remote instance back to<br>> it<br>><br>> src/box/applier.cc | 3 ++-<br>> src/box/box.cc | 13 +++++++++---<br>> src/box/box.h | 3 +++<br>> src/box/iproto_constants.h | 1 +<br>> src/box/relay.cc | 9 ++++++++-<br>> src/box/relay.h | 3 ++-<br>> src/box/wal.c | 17 +++++++++++++---<br>> src/box/xrow.c | 41 +++++++++++++++++++++++++++++++++++---<br>> src/box/xrow.h | 34 +++++++++++++++++++++----------<br>> 9 files changed, 101 insertions(+), 23 deletions(-)<br>></div></div></div></div></div></blockquote><div> </div></BODY></HTML>