Hi Bingshuai,
Thanks for working on it. Confirmed it's a real issue.
I've read your patch and one comment:
The path you added might be redundant if the top-level transaction is being aborted,
right? Can we export the cleanup part outside ReorderBufferAbort()? It does the
actual task only if needed.
Best regards,
Hayato Kuroda
FUJITSU LIMITED