Add CHECK_FOR_INTERRUPTS while decoding changes.

Amit Kapila · Amit Kapila · commit 4985a45917c8 · 2022-08-23T09:10:28.000+05:30
While decoding changes in a loop, if we skip all the changes there is no CFI making the loop uninterruptible. Reported-by: Whale Song and Andrey Borodin Bug: 17580 Author: Masahiko Sawada Reviwed-by: Amit Kapila Backpatch-through: 10 Discussion: https://postgr.es/m/17580-849c1d5b6d7eb422@postgresql.org Discussion: https://postgr.es/m/B319ECD6-9A28-4CDF-A8F4-3591E0BF2369@yandex-team.ru
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c
@@ -1577,6 +1577,8 @@ ReorderBufferCommit(ReorderBuffer *rb, TransactionId xid,
 			Relation	relation = NULL;
 			Oid			reloid;
 
+			CHECK_FOR_INTERRUPTS();
+
 			switch (change->action)
 			{
 				case REORDER_BUFFER_CHANGE_INTERNAL_SPEC_CONFIRM:

Original file line number	Diff line number	Diff line change
`@@ -1577,6 +1577,8 @@ ReorderBufferCommit(ReorderBuffer *rb, TransactionId xid,`
`1577`	`1577`	`Relation relation = NULL;`
`1578`	`1578`	`Oid reloid;`
`1579`	`1579`
	`1580`	`+ CHECK_FOR_INTERRUPTS();`
	`1581`	`+`
`1580`	`1582`	`switch (change->action)`
`1581`	`1583`	`{`
`1582`	`1584`	`case REORDER_BUFFER_CHANGE_INTERNAL_SPEC_CONFIRM:`