Hi,
Thanks for pointing it out!
Here are the new version patches with a test case.
------------------ Original ------------------
From: "Bertrand Drouvot" <bertranddrouvot.pg@gmail.com>;
Date: Wed, Aug 7, 2024 06:33 PM
To: "cca5507"<cca5507@qq.com>;
Cc: "Michael Paquier"<michael@paquier.xyz>;"pgsql-hackers"<pgsql-hackers@lists.postgresql.org>;
Subject: Re: Historic snapshot doesn't track txns committed in BUILDING_SNAPSHOT state
Hi,
On Mon, Jun 10, 2024 at 10:04:31PM +0800, cca5507 wrote:
> Thank you for reply!I am trying to fix it. This patch (pass check-world) will track txns
> committed in BUILDING_SNAPSHOT state and can fix this bug.
Thanks for the report and the patch!
I did not look at the patch in detail but I can see that it does no contain a test
related to this issue.
Would you mind to add a test in say contrib/test_decoding? (see catalog_change_snapshot
for example).
FWIW, to ease the writing of the test, I think you don't need pub/sub to produce
the issue. I think you can reproduce with a single instance and multiple sessions
: replace in your repro the "(sub)create subscription" by a "logical slot creation"
and finish the test by "pg_logical_slot_get_changes" on the slot created above.
Regards,
--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
