| From: | Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com> |
|---|---|
| To: | Ranier Vilela <ranier(dot)vf(at)gmail(dot)com> |
| Cc: | Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Maxim Orlov <m(dot)orlov(at)postgrespro(dot)ru>, Greg Nancarrow <gregn4422(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Pengchengliu <pengchengliu(at)tju(dot)edu(dot)cn>, Andres Freund <andres(at)anarazel(dot)de> |
| Subject: | Re: Parallel scan with SubTransGetTopmostTransaction assert coredump |
| Date: | 2021-06-04 15:07:02 |
| Message-ID: | CALT9ZEHh6AiK4=Dh1XwBfJ7Cj1ru4Uzpj892La9KDPcFzjXD0Q@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
>
> Please, avoid using decimal based values.
>
> 128 is multiple of 64.
>
It's true that 128 is better to use than 120 but the main problem is not in
the value but in the fact we never get
CurrentRunningXacts->subxid_overflow = suboverflowed; with value more than
120. This solves the problem but it doesn't seem the right way to fix the
issue. Instead it's better to process suboverflowed state which is legit
itself not resulting getting the crash on the Assert. So the discussion of
"better" value doesn't seem related to the problem. It is for demonstration
only.
--
Best regards,
Pavel Borisov
Postgres Professional: http://postgrespro.com <http://www.postgrespro.com>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Aleksander Alekseev | 2021-06-04 15:09:58 | Re: Add ZSON extension to /contrib/ |
| Previous Message | Robert Haas | 2021-06-04 14:56:12 | Re: Race condition in recovery? |