• Fujii Masao's avatar
    Use standard SIGTERM signal handler die() in test_shm_mq worker. · ef848f4a
    Fujii Masao authored
    Previously test_shm_mq worker used the stripped-down version of die()
    as the SIGTERM signal handler. This commit makes it use die(), instead,
    to simplify the code.
    
    In terms of the code, the difference between die() and the stripped-down
    version previously used is whether the signal handler directly may call
    ProcessInterrupts() or not. But this difference doesn't exist in
    a background worker because, in bgworker, DoingCommandRead flag will
    never be true and die() will never call ProcessInterrupts() directly.
    Therefore test_shm_mq worker can safely use die(), like other bgworker
    proceses (e.g., logical replication apply launcher or autoprewarm worker)
    currently do.
    
    Thanks to Craig Ringer for the report and investigation of the issue.
    
    Author: Bharath Rupireddy
    Reviewed-by: Fujii Masao
    Discussion: https://postgr.es/m/CAGRY4nxsAe_1k_9g5b47orA0S011iBoHsXHFMH7cg7HV0O1bwQ@mail.gmail.com
    ef848f4a
worker.c 6.28 KB