On Fri, Apr 21, 2023 at 12:30 PM vignesh C <vignes...@gmail.com> wrote: > > On Fri, 21 Apr 2023 at 01:49, Robert Haas <robertmh...@gmail.com> wrote: > > > > On Thu, Apr 20, 2023 at 1:08 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > Pushed. I noticed that we didn't display this new subscription option > > > 'password_required' in \dRs+: > > > > > > postgres=# \dRs+ > > > > > > List of subscriptions > > > Name | Owner | Enabled | Publication | Binary | Streaming | > > > Two-phase commit | Disable on error | Origin | Run as Owner? | > > > Synchronous commit | Conninfo | Skip LSN > > > > > > Is that intentional? Sorry, if it was discussed previously because I > > > haven't followed this discussion in detail. > > > > No, I don't think that's intentional. I just didn't think about it. > > Here is a patch to display Password required with \dRs+ command. Also > added one test to describe subscription when password_required is > false, as all the existing tests were there only for password_required > as true. >
LGTM. Let's see if Robert or others have any comments, otherwise, I'll push this early next week. -- With Regards, Amit Kapila.