From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <vdavydov.dev@gmail.com>
Date: Fri, 1 Mar 2019 12:50:33 +0300
From: Vladimir Davydov <vdavydov.dev@gmail.com>
Subject: Re: [tarantool-patches] [PATCH v5 2/2] Transaction support for
 applier
Message-ID: <20190301095033.4phdr3vemdlwtp6k@esperanza>
References: <cover.1550762885.git.georgy@tarantool.org>
 <71831a4e0a73082d2f0af83ed670abe573fccc97.1550762885.git.georgy@tarantool.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <71831a4e0a73082d2f0af83ed670abe573fccc97.1550762885.git.georgy@tarantool.org>
To: Georgy Kirichenko <georgy@tarantool.org>
Cc: tarantool-patches@freelists.org
List-ID: <tarantool-patches.dev.tarantool.org>

On Thu, Feb 21, 2019 at 06:29:17PM +0300, Georgy Kirichenko wrote:
> Applier fetch incoming rows to form a transaction and then apply it.
> In case of  replication all local changes moved to an journal entry
> tail to form a separate transaction (like autonomous transaction)
> to be able to replicate changes back. Applier assumes that transactions
> could not be mixed in a replication stream.

Discussed with Kostja. Here's what he thinks we need to do to commit
this patch:

 1. Get rid of xstream in applier. Should be done in a separate patch.

 2. Instead of using obuf for requests and ibuf for headers, we should
    copy those to fiber->gc.