From: Cyrill Gorcunov via Tarantool-patches <tarantool-patches@dev.tarantool.org> To: tml <tarantool-patches@dev.tarantool.org> Cc: Vladislav Shpilevoy <v.shpilevoy@tarantool.org> Subject: [Tarantool-patches] [PATCH v19 0/3] qsync: implement packet filtering (part 1) Date: Thu, 30 Sep 2021 12:44:42 +0300 [thread overview] Message-ID: <20210930094445.316694-1-gorcunov@gmail.com> (raw) Please take a look once time permit. v19 (by Vlad): - do not modify box_issue_promote and demote (while they are still simply utter code duplication but whatever) - make txn_limbo_process being void - make txn_limbo_process_begin/commit/rollback being void - the real processing of request under the lock named as txn_limbo_process_core - testcase completely reworked (kudos to SergeP) - note that if we import test to the master branch without ordering pass it will fire assertion - dropped off debug info from box.info interface Guys, also I noted that the test while being passed pretty fine locally sometimes doesn't pass on CI due to timeout in waiting record {2} propagation, so any advices on the test is quite welcome! branch gorcunov/gh-6036-rollback-confirm-19 issue https://github.com/tarantool/tarantool/issues/6036 previous series https://lists.tarantool.org/tarantool-patches/20210922130535.79479-1-gorcunov@gmail.com/ Cyrill Gorcunov (3): latch: add latch_is_locked helper qsync: order access to the limbo terms test: add gh-6036-qsync-order test src/box/applier.cc | 12 +- src/box/box.cc | 15 +- src/box/txn_limbo.c | 17 +- src/box/txn_limbo.h | 53 ++++- src/lib/core/latch.h | 11 + test/replication/gh-6036-qsync-master.lua | 1 + test/replication/gh-6036-qsync-node.lua | 35 +++ test/replication/gh-6036-qsync-order.result | 207 ++++++++++++++++++ test/replication/gh-6036-qsync-order.test.lua | 95 ++++++++ test/replication/gh-6036-qsync-replica1.lua | 1 + test/replication/gh-6036-qsync-replica2.lua | 1 + test/replication/suite.cfg | 1 + test/replication/suite.ini | 2 +- 13 files changed, 433 insertions(+), 18 deletions(-) create mode 120000 test/replication/gh-6036-qsync-master.lua create mode 100644 test/replication/gh-6036-qsync-node.lua create mode 100644 test/replication/gh-6036-qsync-order.result create mode 100644 test/replication/gh-6036-qsync-order.test.lua create mode 120000 test/replication/gh-6036-qsync-replica1.lua create mode 120000 test/replication/gh-6036-qsync-replica2.lua base-commit: 33830978870bbe8cdc3c23fa06ba0f141a960124 -- 2.31.1
next reply other threads:[~2021-09-30 9:44 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-09-30 9:44 Cyrill Gorcunov via Tarantool-patches [this message] 2021-09-30 9:44 ` [Tarantool-patches] [PATCH v19 1/3] latch: add latch_is_locked helper Cyrill Gorcunov via Tarantool-patches 2021-09-30 9:44 ` [Tarantool-patches] [PATCH v19 2/3] qsync: order access to the limbo terms Cyrill Gorcunov via Tarantool-patches 2021-10-01 12:14 ` Serge Petrenko via Tarantool-patches 2021-10-01 12:31 ` Cyrill Gorcunov via Tarantool-patches 2021-10-01 12:37 ` Serge Petrenko via Tarantool-patches 2021-10-04 21:53 ` Cyrill Gorcunov via Tarantool-patches 2021-10-05 13:25 ` Serge Petrenko via Tarantool-patches 2021-10-05 21:52 ` Cyrill Gorcunov via Tarantool-patches 2021-09-30 9:44 ` [Tarantool-patches] [PATCH v19 3/3] test: add gh-6036-qsync-order test Cyrill Gorcunov via Tarantool-patches 2021-10-01 12:30 ` Serge Petrenko via Tarantool-patches 2021-10-04 21:16 ` Cyrill Gorcunov via Tarantool-patches 2021-10-05 13:55 ` Serge Petrenko via Tarantool-patches 2021-10-05 22:26 ` Cyrill Gorcunov via Tarantool-patches 2021-10-05 22:32 ` Cyrill Gorcunov via Tarantool-patches 2021-10-06 7:06 ` Cyrill Gorcunov via Tarantool-patches
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20210930094445.316694-1-gorcunov@gmail.com \ --to=tarantool-patches@dev.tarantool.org \ --cc=gorcunov@gmail.com \ --cc=v.shpilevoy@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v19 0/3] qsync: implement packet filtering (part 1)' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox