Tarantool development patches archive
 help / color / mirror / Atom feed
From: Cyrill Gorcunov <gorcunov@gmail.com>
To: tml <tarantool-patches@dev.tarantool.org>
Cc: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>
Subject: [Tarantool-patches] [PATCH 0/5] qsync: write confirm/rollback without txn engine
Date: Wed, 22 Jul 2020 18:33:54 +0300	[thread overview]
Message-ID: <20200722153359.134718-1-gorcunov@gmail.com> (raw)

Vlad, take a look please once time permit. Note the series is on top
of your series "[PATCH 0/2] Make txn_commit() simpler"

I think in our wal engine we should go further and eliminate completion
calls for synchronous writes completely (internally WAL could setup
completion to fiber_wakeup and reuse async engine) but from API it would
not call completion associated with transactions ever. But such redesign
is defenitely for other series.

Anyway in this series we make a first approach to not use txn engine when
we have to simply write confirm/rollback record into a journal.

issue https://github.com/tarantool/tarantool/issues/5129
branch gorcunov/gh-5129-journal

Cyrill Gorcunov (5):
  journal: drop redundant declaration
  wal: bind asynchronous write completion to an entry
  journal: add journal_entry_create helper
  qsync: implement direct write of confirm/rollback into a journal
  qsync: fix release build

 src/box/box.cc             |  14 ++---
 src/box/iproto_constants.h |  24 +++++++++
 src/box/journal.c          |   8 ++-
 src/box/journal.h          |  39 ++++++++++----
 src/box/txn.c              |   2 +-
 src/box/txn_limbo.c        | 101 ++++++++++++++++++++-----------------
 src/box/vy_log.c           |   2 +-
 src/box/wal.c              |  20 ++++----
 src/box/wal.h              |   4 +-
 src/box/xrow.c             |  46 ++++-------------
 src/box/xrow.h             |  31 ++++--------
 11 files changed, 151 insertions(+), 140 deletions(-)

-- 
2.26.2

             reply	other threads:[~2020-07-22 15:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-22 15:33 Cyrill Gorcunov [this message]
2020-07-22 15:33 ` [Tarantool-patches] [PATCH 1/5] journal: drop redundant declaration Cyrill Gorcunov
2020-07-22 15:33 ` [Tarantool-patches] [PATCH 2/5] wal: bind asynchronous write completion to an entry Cyrill Gorcunov
2020-07-22 15:33 ` [Tarantool-patches] [PATCH 3/5] journal: add journal_entry_create helper Cyrill Gorcunov
2020-07-22 15:33 ` [Tarantool-patches] [PATCH 4/5] qsync: implement direct write of confirm/rollback into a journal Cyrill Gorcunov
2020-07-22 15:41   ` Cyrill Gorcunov
2020-07-22 15:33 ` [Tarantool-patches] [PATCH 5/5] qsync: fix release build Cyrill Gorcunov
2020-07-23 11:41 ` [Tarantool-patches] [PATCH 0/5] qsync: write confirm/rollback without txn engine Cyrill Gorcunov

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=20200722153359.134718-1-gorcunov@gmail.com \
    --to=gorcunov@gmail.com \
    --cc=tarantool-patches@dev.tarantool.org \
    --cc=v.shpilevoy@tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH 0/5] qsync: write confirm/rollback without txn engine' \
    /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