Re: Identify missing publications from publisher while create/alter subscription.
vignesh C <vignesh21@gmail.com>
From: vignesh C <vignesh21@gmail.com>
To: Ashutosh Sharma <ashu.coek88@gmail.com>
Cc: Euler Taveira <euler@eulerto.com>,
Jaime Casanova <jcasanov@systemguards.com.ec>, Dilip Kumar <dilipbalaut@gmail.com>, japin <japinli@hotmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Date: 2022-02-13T14:02:30Z
Lists: pgsql-hackers
On Thu, Feb 10, 2022 at 3:15 PM Ashutosh Sharma <ashu.coek88@gmail.com> wrote: > > On Wed, Feb 9, 2022 at 11:53 PM Euler Taveira <euler@eulerto.com> wrote: > > > > On Wed, Feb 9, 2022, at 12:06 PM, Ashutosh Sharma wrote: > > > > Just wondering if we should also be detecting the incorrect conninfo > > set with ALTER SUBSCRIPTION command as well. See below: > > > > -- try creating a subscription with incorrect conninfo. the command fails. > > postgres=# create subscription sub1 connection 'host=localhost > > port=5490 dbname=postgres' publication pub1; > > ERROR: could not connect to the publisher: connection to server at > > "localhost" (::1), port 5490 failed: Connection refused > > Is the server running on that host and accepting TCP/IP connections? > > connection to server at "localhost" (127.0.0.1), port 5490 failed: > > Connection refused > > Is the server running on that host and accepting TCP/IP connections? > > > > That's because by default 'connect' parameter is true. > > > > So can we use this option with the ALTER SUBSCRIPTION command. I think > we can't, which means if the user sets wrong conninfo using ALTER > SUBSCRIPTION command then we don't have the option to detect it like > we have in case of CREATE SUBSCRIPTION command. Since this thread is > trying to add the ability to identify the wrong/missing publication > name specified with the ALTER SUBSCRIPTION command, can't we do the > same for the wrong conninfo? I felt this can be extended once this feature is committed. Thoughts? Regards, Vignesh
Commits
-
Raise a WARNING for missing publications.
- 8f2e2bbf1453 15.0 landed