From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
---|---|
To: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Move replication slot structures/enums/macros to a new header file for better usability by external tools/modules |
Date: | 2022-02-08 20:46:00 |
Message-ID: | CAHut+Ps9R2VmNGu4Qxt6n4wyUyCYkyeenAhOo9iZ3bHT=ykDjw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Feb 7, 2022 at 4:22 PM Bharath Rupireddy
<bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
>
> Hi,
>
> While working on pg_replslotdata tool [1], it was observed that some
> of the replication slot structures/enums/macros such as
> ReplicationSlotPersistentData, ReplicationSlotPersistency,
> ReplicationSlotOnDisk, ReplicationSlotOnDiskXXXX etc. are currently in
> replication/slot.h and replication/slot.c. This makes the replication
> slot on disk data structures unusable by the external tools/modules.
> How about moving these structures to a new header file called
> slot_common.h as attached in the patch here?
>
> Thoughts?
>
> PS: I'm planning to have the tool separately, as it was rejected to be in core.
>
> [1] https://www.postgresql.org/message-id/CALj2ACW0rV5gWK8A3m6_X62qH%2BVfaq5hznC%3Di0R5Wojt5%2Byhyw%40mail.gmail.com
>
> Regards,
> Bharath Rupireddy.
Recently I was also looking to add some new enums but I found it
was difficult to find any good place to put them where they could be
shared by the replication code and the pg_recvlogical tool.
So +1 to your suggestion to have a common header, but I wonder can it
have a more generic name (e.g. repl_common.h? ...) since the
stuff I wanted to put there was not really "slot" related.
------
Kind Regards,
Peter Smith.
Fujitsu Australia.
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-02-08 21:10:51 | Re: Fix BUG #17335: Duplicate result rows in Gather node |
Previous Message | Tom Lane | 2022-02-08 20:42:01 | Re: Refactoring the regression tests for more independence |