Re: [19] CREATE SUBSCRIPTION ... SERVER

From: Jeff Davis <pgsql(at)j-davis(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Shlok Kyal <shlok(dot)kyal(dot)oss(at)gmail(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Joe Conway <mail(at)joeconway(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [19] CREATE SUBSCRIPTION ... SERVER
Date: 2026-03-10 14:23:46
Message-ID: 9b4795c19425f67fab975e5487f5b43567ca40a3.camel@j-davis.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 2026-03-09 at 11:53 +0530, Amit Kapila wrote:
> > +# Replicate the changes without columns
> > +$node_publisher->safe_psql('postgres', "CREATE TABLE
> > tab_no_col()");
> > +$node_publisher->safe_psql('postgres',
> > + "INSERT INTO tab_no_col default VALUES");
> >
> > I don't see a subscriber-side table or verification code to verify
> > the
> > above test.
> >
>
> I see that the committed version (8185bb5347) has this part of the
> test, isn't that test incomplete, if not, tell me what am I missing?

In 8185bb5347, contrib/postgres_fdw/t/010_subscription.pl has:

...
# Setup structure on subscriber
$node_subscriber->safe_psql('postgres', "CREATE EXTENSION
postgres_fdw");
$node_subscriber->safe_psql('postgres', "CREATE TABLE tab_ins (a int,
b int)");
...
$result =
$node_subscriber->safe_psql('postgres', "SELECT count(*) FROM (SELECT
f.b = l.b as match FROM tab_ins l, f_tab_ins f WHERE l.a = f.a) WHERE
match");
is($result, qq(1050), 'check that inserted data was copied to
subscriber');
...

which creates the subscriber-side table and verifies the result. If I
change 1050 -> 1051, then the test fails, so I think it's functioning.

Perhaps I don't understand the question?

> It seems I have sent this message after you have committed the last
> version.

Yes, thank you, I will address those shortly.

Regards,
Jeff Davis

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2026-03-10 14:31:44 Re: [PATCH] Simplify trivial shmem size calculations
Previous Message Greg Sabino Mullane 2026-03-10 14:23:21 Re: psql's 001_basic.pl test could fail on very slow machines