← Back to team overview

maria-developers team mailing list archive

Re: c5ce597f06a: MDEV-31043 ER_KEY_NOT_FOUND upon concurrent ALTER and transaction


Hello Sergei!

On Wed, 3 May 2023 at 15:29, Sergei Golubchik <serg@xxxxxxxxxxx> wrote:

> you've lost the useful comment about STMT_END.
Okay. I just thought that from the reader's perspective it doesn't look
important. But you are the reader, and you say it's important. Bringing

> and you flush events for rollback too that seems like a waste.

I can avoid it, but then we'll have to clear the pending event. Ok i'll do
it, i think i didn't see the need to optimize these few bits of rollback.

> I personally would've kept the old structure of if()'s.
 I like the new one more, as it resembles the symmetry between storing and
restoring binlog's position,
and besides it has fewer nesting.

See my new commit b804bcbc6
<here: https://github.com/MariaDB/server/commit/b804bcbc64dfc0bc7b244eaebbd64c987bdb5a93>,
or the diff with the old one, here
Yours truly,
Nikita Malyavin

Follow ups