• Tom Lane's avatar
    Further second thoughts about idle_session_timeout patch. · b8d0cda5
    Tom Lane authored
    On reflection, the order of operations in PostgresMain() is wrong.
    These timeouts ought to be shut down before, not after, we do the
    post-command-read CHECK_FOR_INTERRUPTS, to guarantee that any
    timeout error will be detected there rather than at some ill-defined
    later point (possibly after having wasted a lot of work).
    
    This is really an error in the original idle_in_transaction_timeout
    patch, so back-patch to 9.6 where that was introduced.
    b8d0cda5
postgres.c 131 KB