On Mon, 18 Dec 2023 at 17:40, Andrey M. Borodin <x4...@yandex-team.ru> wrote: >> On 18 Dec 2023, at 14:32, Japin Li <japi...@hotmail.com> wrote: >> >> >> Thanks for updating the patch > > Sorry for the noise, but commitfest bot found one more bug in handling > statement timeout. PFA v11. >
On Windows, there still have an error: diff -w -U3 C:/cirrus/src/test/isolation/expected/timeouts.out C:/cirrus/build/testrun/isolation/isolation/results/timeouts.out --- C:/cirrus/src/test/isolation/expected/timeouts.out 2023-12-18 10:22:21.772537200 +0000 +++ C:/cirrus/build/testrun/isolation/isolation/results/timeouts.out 2023-12-18 10:26:08.039831800 +0000 @@ -103,24 +103,7 @@ 0 (1 row) -step stt2_check: SELECT 1; -FATAL: terminating connection due to transaction timeout -server closed the connection unexpectedly +PQconsumeInput failed: server closed the connection unexpectedly This probably means the server terminated abnormally before or while processing the request. -step itt4_set: SET idle_in_transaction_session_timeout = '1ms'; SET statement_timeout = '10s'; SET lock_timeout = '10s'; SET transaction_timeout = '10s'; -step itt4_begin: BEGIN ISOLATION LEVEL READ COMMITTED; -step sleep_there: SELECT pg_sleep(0.01); -pg_sleep --------- - -(1 row) - -step stt3_check_itt4: SELECT count(*) FROM pg_stat_activity WHERE application_name = 'isolation/timeouts/itt4' <waiting ...> -step stt3_check_itt4: <... completed> -count ------ - 0 -(1 row) - -- Regrads, Japin Li ChengDu WenWu Information Technology Co., Ltd.