Remove redundant SetLatch() calls in interrupt handling functions Interrupt handling functions (e.g., HandleCatchupInterrupt(), HandleParallelApplyMessageInterrupt()) are called only by procsignal_sigusr1_handler(), which already calls SetLatch() for the current process at the end of its processing. Therefore, these interrupt handling functions do not need to call SetLatch() themselves.
However, previously, some of these functions redundantly called SetLatch(). This commit removes those unnecessary calls. While duplicate SetLatch() calls are redundant, they are harmless, so this change is not backpatched. Author: Bharath Rupireddy <[email protected]> Reviewed-by: Bertrand Drouvot <[email protected]> Reviewed-by: Fujii Masao <[email protected]> Reviewed-by: Chao Li <[email protected]> Reviewed-by: Dilip Kumar <[email protected]> Discussion: https://postgr.es/m/calj2acwd5apddj6cd885wwj6lquyu_g81c4gor4xsodv1x-...@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/57706799186aa7971b78cab8911fb2604833de0f Modified Files -------------- src/backend/access/transam/parallel.c | 2 +- src/backend/commands/async.c | 3 +-- src/backend/replication/logical/applyparallelworker.c | 2 +- src/backend/replication/walsender.c | 2 ++ src/backend/storage/ipc/sinval.c | 3 +-- 5 files changed, 6 insertions(+), 6 deletions(-)
