[tarantool-patches] [PATCH 2/4] vinyl: don't abort transactions that modify only local spaces for ro

Vladimir Davydov vdavydov.dev at gmail.com
Mon Mar 25 12:30:02 MSK 2019


On Mon, Mar 25, 2019 at 11:58:18AM +0300, Георгий Кириченко wrote:
> On Monday, March 25, 2019 11:13:51 AM MSK Vladimir Davydov wrote:
> > On Mon, Mar 25, 2019 at 08:27:28AM +0300, Георгий Кириченко wrote:
> > > I think it is out of scope of the task.
> > 
> > So what? This was a known issue when the switch_to_ro was introduced.
> > Better fix it now than never, especially counting the fact that it's
> > a low hanging fruit after patch 1.
> So let it be in a separate patch with assigned ticket.

Why? It depends on the previous patch.

I ask to move a patch out of a series only if there's a great chance to
commit it before the rest of the series.

Filing a ticket when the patch is ready looks like excessive
bureaucracy to me. In fact, it's just a follow-up for #4016.



More information about the Tarantool-patches mailing list