Re: src/bin/pg_upgrade/t/004_subscription.pl test comment fix - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: src/bin/pg_upgrade/t/004_subscription.pl test comment fix
Date
Msg-id 202401310848.d6jovn6426uo@alvherre.pgsql
Whole thread Raw
In response to src/bin/pg_upgrade/t/004_subscription.pl test comment fix  (Peter Smith <smithpb2250@gmail.com>)
Responses Re: src/bin/pg_upgrade/t/004_subscription.pl test comment fix
List pgsql-hackers
How about rewording it more extensively?  It doesn't read great IMO.
I would use something like

# In the upgraded instance, the running status and failover option of the
# subscription with the failover option should have been preserved; the other
# should not.
# So regress_sub1 should still have subenabled,subfailover set to true,
# while regress_sub2 should have both set to false.

I think the symmetry between the two lines confuses more than helps.
It's not a huge thing but since we're editing anyway, why not?

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/
"The saddest aspect of life right now is that science gathers knowledge faster
 than society gathers wisdom."  (Isaac Asimov)



pgsql-hackers by date:

Previous
From: Nazir Bilal Yavuz
Date:
Subject: Re: 003_extrafiles.pl test fails on Windows with the newer Perl versions
Next
From: Thomas Munro
Date:
Subject: Re: Extending SMgrRelation lifetimes