Re: Incorrect check in 037_except.pl? - Mailing list pgsql-hackers

From Ayush Tiwari
Subject Re: Incorrect check in 037_except.pl?
Date
Msg-id CAJTYsWUk+5m8jjGrxKT5zMZT2O2DHQLARXYzGy-0tzbM2yCUqA@mail.gmail.com
Whole thread
In response to Incorrect check in 037_except.pl?  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
Hi,

On Wed, 22 Jul 2026 at 18:29, Fujii Masao <masao.fujii@gmail.com> wrote:
Hi,

    $result =
      $node_publisher->safe_psql('postgres', "SELECT * FROM tab1 ORDER BY a");
    is( $result, qq(1
    2),
      "check replication of a table in the EXCEPT clause of one
publication but included by another"
    );

In 037_except.pl, this query is executed on the publisher. But,
if the intent is to verify that rows inserted on the publisher are
replicated to the subscriber, shouldn't this query be executed on
the subscriber instead? IOW, should "$node_publisher" here be
"$node_subscriber"? Or am I missing something?
 
Yeah you're right, it should likely be the subscriber.

Same issue appears twice (the "FOR TABLE" and "FOR ALL TABLES" cases);
both need the change.

Regards,
Ayush

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: convert various variables to atomics
Next
From: Amit Kapila
Date:
Subject: Re: Fix "unexpected logical decoding status change" error; from concurrent logical decoding activation