Re: Logical replication 'invalid memory alloc request size 1585837200' after upgrading to 17.5
Masahiko Sawada <sawada.mshk@gmail.com>
From: Masahiko Sawada <sawada.mshk@gmail.com>
To: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>
Cc: "pgsql-bugs@lists.postgresql.org" <pgsql-bugs@lists.postgresql.org>,
Amit Kapila <amit.kapila16@gmail.com>, Duncan Sands <duncan.sands@deepbluecap.com>, vignesh C <vignesh21@gmail.com>, Alexander Lakhin <exclusion@gmail.com>
Date: 2025-06-21T00:17:32Z
Lists: pgsql-bugs
On Fri, Jun 20, 2025 at 5:21 PM Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com> wrote: > > Dear hackers, > > I've analyzed the issue, and this can always happen when --DCLOBBER_CACHE_ALWAYS is > set on PG13. My suggestion is to remove the latter part of test on this branch. > > In the failed workload, we tested the case that one long transaction inserts a > tuple after the concurrent transaction does ALTER PUBLICATION ADD TABLE. > For PG14+ the change can be published, and PG13 it cannot be replicated because > the distributed inval messages cannot be executed during the transaction. That's > why we expected no results were output. > > So what if the -DDCLOBBER_CACHE_ALWAYS is set for the workload? relsync cache > can be discarded very frequently and backend process always read the system > catalog when it processes their changes. This means decoder can recognize that the > pg_publication has been updated and it can publish changes based on the altered > publication information. This behavior is opposite from normal. > > To fix the test failure, I suggest to just remove the case. Insert-after-commit > case has already been tested by above part of this file, so no need to do others. > > Attached file does accordingly. Thank you for providing the patch. I'll look at the issue and the patch in depth but it sounds like a reasonable solution. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Fix cache-dependent test failures in logical decoding.
- 87819f766f37 13.22 landed
-
Fix re-distributing previously distributed invalidation messages during logical decoding.
- d87d07b7ad3b 18.0 landed
- 45c357e0e85d 17.6 landed
- b2ae077205e1 16.10 landed
- fc0fb77c550f 15.14 landed
- 983b3636259b 14.19 landed
- 1230be12f086 13.22 landed
-
Fix data loss in logical replication.
- 247ee94150b6 13.21 cited
- 4909b38af034 18.0 cited