Re: [PATCH] Explain what the default output_plugin_libraries do - Mailing list pgsql-hackers

From Chao Li
Subject Re: [PATCH] Explain what the default output_plugin_libraries do
Date
Msg-id BFD7E925-B4A6-4DE0-AD04-BB3A47089628@gmail.com
Whole thread
In response to Re: [PATCH] Explain what the default output_plugin_libraries do  (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>)
Responses Re: [PATCH] Explain what the default output_plugin_libraries do
List pgsql-hackers

> On Sep 22, 2026, at 06:45, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> wrote:
>
> Hi,
>
> On Mon, Sep 21, 2026 at 9:02 AM Jacob Champion
> <jacob.champion@enterprisedb.com> wrote:
>>
>>>> Okay. Any thoughts on Bharath's note on "publication slot creation" above?
>>>
>>> I could not find the term "publication slot" in the existing doc, so I prefer
>>> his idea. Another one I came up with to just simplify:
>>>
>>> ```
>>> Removing from output_plugin_libraries will cause future logical replication to
>>> fail with an error. (Beware that existing replication tools may not degrade gracefully
>>> if the server is configured in this way.)
>>
>> But only built-in logical replication will fail. How about
>>
>>             Core plugin for logical replication. Removing
>> <literal>pgoutput</literal>
>>             from <literal>output_plugin_libraries</literal> will cause future
>>             subscription connections, and maintenance of the replication slots
>>             for subscriptions, to fail with an error. (Beware that ...
>>
>> ?
>
> This part works for me. Thanks.
>

This also looks good to me.

I have the other nitpick:
```
          <productname>PostgreSQL</productname> distribution.
+         (<xref linkend="guc-output-plugin-libraries-builtins"/> documents the
+         behavior of the system if they are removed from this list.)
         </para>
```

I tried to build the HTML page, and see that <xref linkend="guc-output-plugin-libraries-builtins”/> is rendered as
“Table19.2”, so it looks like “(Table 19.2 documents ….)”, which looks a little awkward. How about not using
parentheses,and just say something like: See <xref linkend="guc-output-plugin-libraries-builtins"/> for the effects of
removingthese plugins from the list. 

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/







pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Add counted_by attribute
Next
From: Nikolay Samokhvalov
Date:
Subject: Re: Fix "unexpected logical decoding status change" error; from concurrent logical decoding activation