From: | "Ideriha, Takeshi" <ideriha(dot)takeshi(at)jp(dot)fujitsu(dot)com> |
---|---|
To: | 'Tatsuo Ishii' <ishii(at)sraoss(dot)co(dot)jp> |
Cc: | "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | RE: idle-in-transaction timeout error does not give a hint |
Date: | 2018-12-03 01:39:54 |
Message-ID: | 4E72940DA2BF16479384A86D54D0988A6F3C1E24@G01JPEXMBKW04 |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>From: Tatsuo Ishii [mailto:ishii(at)sraoss(dot)co(dot)jp]
>
>Hi Ideriha-san,
>
>>>>> Hi, it makes sense to me. One can submit transaction again same as
>>>>> other cases you mentioned.
>>>>>
>>>>> I didn't attach the patch but according to my simple experiment in
>>>>> psql the output would become the following:
>>>>>
>>>>> FATAL: terminating connection due to idle-in-transaction timeout
>>>>> HINT: In a moment you should be able to reconnect to the
>>>>> database and repeat your command.
>>>>
>>>>Alternative HINT message would be something like:
>>>>
>>>>HINT: In a moment you should be able to reconnect to the
>>>> database and restart your transaction.
>>>>
>>>>This could make the meaning of the error (transaction aborted)
>>>>cleaner and might give a better suggestion to the user.
>>>
>>> Agreed. Changing "command" to "transaction" seems more accurate.
>>> People might think only the command they hit is not sent but
>>> transaction is still alive though it's of course unnatural that transaction is alive after
>connection is terminated.
>Please find attached patch which addresses the point above.
Thank you for the update. It looks good to me on this point.
Are you planning to change other similar messages?
>BTW, would you like to be added to the CF item as a reviewer?
Yeah, I've already added myself as a review.
Regards,
Takeshi Ideriha
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Gierth | 2018-12-03 01:47:11 | Re: [proposal] Add an option for returning SQLSTATE in psql error message |
Previous Message | Michael Paquier | 2018-12-03 00:48:43 | Re: Online verification of checksums |