Use correct connection for cancellation in frontend's parallel slots
authorMichael Paquier <michael@paquier.xyz>
Sat, 27 Aug 2022 06:21:31 +0000 (15:21 +0900)
committerMichael Paquier <michael@paquier.xyz>
Sat, 27 Aug 2022 06:21:31 +0000 (15:21 +0900)
commit52144b6fcd626bb7dbe9fe282329c0174620a34b
tree8c0abdd16f4730a3d4820c69b73f685ffe9186ac
parente890ce7a4feb9d72cd502d52c5a4c4d853801974
Use correct connection for cancellation in frontend's parallel slots

While waiting for slots to become available in wait_on_slots() in
parallel_slot.c, the cancellation always relied on the first connection
in the set to do the job.  This could cause problems when this slot's
socket is gone as PQgetCancel() would return NULL in this case.  Rather
than always using the first connection, this changes the logic to use
the first valid connection for the cancellation.

Author: Ranier Vilela
Reviewed-by: Justin Pryzby
Discussion: https://postgr.es/m/CAEudQAokk1h_pUwGXsYS4oVOuf35s1O2o3TXGHpV8=AWikvgHA@mail.gmail.com
Backpatch-through: 14
src/fe_utils/parallel_slot.c