Remove obsolete speculative insert cleanup in ReorderBuffer.

Commit 4daa140a2f introduced proper decoding for speculative aborts. As a
result, the internal state is guaranteed to be clean when a new
speculative insert is encountered. This patch removes the defensive
cleanup code that is no longer reachable.

Author: Antonin Houska <ah@cybertec.at>
Discussion: https://postgr.es/m/23256.1772702981@localhost
This commit is contained in:
Amit Kapila 2026-03-16 10:14:22 +05:30
parent d8879d34b9
commit 5f39698c90

View file

@ -2447,12 +2447,8 @@ ReorderBufferProcessTXN(ReorderBuffer *rb, ReorderBufferTXN *txn,
* CheckTableNotInUse() and locking.
*/
/* clear out a pending (and thus failed) speculation */
if (specinsert != NULL)
{
ReorderBufferFreeChange(rb, specinsert, true);
specinsert = NULL;
}
/* Previous speculative insertion must be aborted */
Assert(specinsert == NULL);
/* and memorize the pending insertion */
dlist_delete(&change->node);