pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Justin Pryzby <pryzby@telsasoft.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: vignesh C <vignesh21@gmail.com>
Cc: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Peter Smith <smithpb2250@gmail.com>
Cc: Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
Cc: Julien Rouhaud <rjuju123@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: pg_upgrade and logical replication
Date: Tue, 13 Feb 2024 15:05:14 -0600
Message-ID: <ZcvZipRoi_kopIpb@pryzbyj2023> (raw)
In-Reply-To: <CAA4eK1+dM0DcFRgaLp3syN0FMM2DAkp8PmtTUdgymx5D0d4y_w@mail.gmail.com>
References: <CAHut+Pst-wJ-2nw90gAof+TbbfQC1ERsVeWvG_HaNH7xqu4Fdg@mail.gmail.com>
	<CALDaNm0U5QsyE0yJxt7rHkH-tO=0Q1Fu=PoueqL6boLGOUXN7g@mail.gmail.com>
	<CAA4eK1K-YGFOmS=Gi-GST0A5Xx4zBsoXn0TebDtZFNKUmjDPww@mail.gmail.com>
	<ZW60c-tpTNB5noL8@paquier.xyz>
	<CALDaNm27+B6hiCS3g3nUDpfwmTaj6YopSY5ovo2=__iOSpkPbA@mail.gmail.com>
	<CAD21AoBN6T+vPQuf_y9vrUX7dhOfmkVjqc4vQwdDozTH_xZa4g@mail.gmail.com>
	<CALDaNm3p16KYd70QKV9qYHqWqQV1G4i2ugaifst9Viwfp_XkpA@mail.gmail.com>
	<CAA4eK1+TqV7Gcsu=Ai2iSy6fH-gi6MKocX+L52_g567b1O5gig@mail.gmail.com>
	<CALDaNm20wZwWcjF+OmwEAa7ugREPKyY0L29ngkug33DhNWz_hA@mail.gmail.com>
	<CAA4eK1+dM0DcFRgaLp3syN0FMM2DAkp8PmtTUdgymx5D0d4y_w@mail.gmail.com>

On Tue, Jan 02, 2024 at 03:58:25PM +0530, Amit Kapila wrote:
> Pushed.

pg_upgrade/t/004_subscription.pl says

|my $mode = $ENV{PG_TEST_PG_UPGRADE_MODE} || '--copy';

..but I think maybe it should not.

When you try to use --link, it fails:
https://cirrus-ci.com/task/4669494061170688

|Adding ".old" suffix to old global/pg_control                 ok
|
|If you want to start the old cluster, you will need to remove
|the ".old" suffix from /tmp/cirrus-ci-build/build/testrun/pg_upgrade/004_subscription/data/t_004_subscription_old_sub_data/pgdata/global/pg_control.old.
|Because "link" mode was used, the old cluster cannot be safely
|started once the new cluster has been started.
|...
|
|postgres: could not find the database system
|Expected to find it in the directory "/tmp/cirrus-ci-build/build/testrun/pg_upgrade/004_subscription/data/t_004_subscription_old_sub_data/pgdata",
|but could not open file "/tmp/cirrus-ci-build/build/testrun/pg_upgrade/004_subscription/data/t_004_subscription_old_sub_data/pgdata/global/pg_control": No such file or directory
|# No postmaster PID for node "old_sub"
|[19:36:01.396](0.250s) Bail out!  pg_ctl start failed

You could rename pg_control.old to avoid that immediate error, but that doesn't
address the essential issue that "the old cluster cannot be safely started once
the new cluster has been started."

-- 
Justin





view thread (221+ messages)  latest in thread

Message-ID: <ZcvZipRoi_kopIpb@pryzbyj2023>
Permalink:  ../ZcvZipRoi_kopIpb@pryzbyj2023/
Also on:    postgresql.org/message-id/ZcvZipRoi_kopIpb@pryzbyj2023

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: pryzby@telsasoft.com, amit.kapila16@gmail.com, vignesh21@gmail.com, sawada.mshk@gmail.com, michael@paquier.xyz, smithpb2250@gmail.com, kuroda.hayato@fujitsu.com, rjuju123@gmail.com, pgsql-hackers@lists.postgresql.org
  Subject: Re: pg_upgrade and logical replication
  In-Reply-To: <ZcvZipRoi_kopIpb@pryzbyj2023>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox