| From: | ocean_li_996 <ocean_li_996(at)163(dot)com> |
|---|---|
| To: | "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com> |
| Cc: | "Masahiko Sawada" <sawada(dot)mshk(at)gmail(dot)com>, "Amit Kapila" <amit(dot)kapila16(at)gmail(dot)com>, "pgsql-bugs(at)lists(dot)postgresql(dot)org" <pgsql-bugs(at)lists(dot)postgresql(dot)org> |
| Subject: | Re:RE: BUG #18055: logical decoding core on AllocateSnapshotBuilder() |
| Date: | 2023-08-20 16:51:30 |
| Message-ID: | 461dd3e6.197a.18a13dbdc8a.Coremail.ocean_li_996@163.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Excited! Thanks for the discussion and the fix!
The bug seems to be solved well. And usage of PG_FINALLY can make it more elegantly. However, for the version lacking of PG_FINALLY, should the patch be written like that:
---
/* free context, call shutdown callback */
FreeDecodingContext(ctx);
ReplicationSlotRelease();
InvalidateSystemCaches();
}
PG_CATCH();
{
+
+ /* free context, call shutdown callback */
+ if (ctx != NULL)
+ FreeDecodingContext(ctx);
+
+ ReplicationSlotRelease();
----
Best regards!
At 2023-08-18 12:21:53, "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com> wrote:
Here is a new version patch which addressed this based on PG15.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David G. Johnston | 2023-08-20 16:58:01 | Re: BUG #18061: The psql -v variable option does not work for the -c command option |
| Previous Message | Tom Lane | 2023-08-20 16:12:21 | Re: BUG #18060: Left joining rows using random() function in join condition doesn't work as expected. |