Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kzNH9-0000uR-Ak for pgsql-hackers@arkaria.postgresql.org; Tue, 12 Jan 2021 17:16:11 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1kzNH8-0005yq-4o for pgsql-hackers@arkaria.postgresql.org; Tue, 12 Jan 2021 17:16:10 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kzNH7-0005yj-U0 for pgsql-hackers@lists.postgresql.org; Tue, 12 Jan 2021 17:16:09 +0000 Received: from momjian.us ([72.94.173.45]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kzNH5-0001k5-P5 for pgsql-hackers@lists.postgresql.org; Tue, 12 Jan 2021 17:16:08 +0000 Received: from bruce by momjian.us with local (Exim 4.92) (envelope-from ) id 1kzNGx-0004CI-Aj; Tue, 12 Jan 2021 12:15:59 -0500 Date: Tue, 12 Jan 2021 12:15:59 -0500 From: Bruce Momjian To: Justin Pryzby Cc: Peter Eisentraut , Tomas Vondra , Tom Lane , Andres Freund , buschmann@nidsa.net, pgsql-hackers@lists.postgresql.org, Noah Misch Subject: Re: pg_upgrade test for binary compatibility of core data types Message-ID: <20210112171559.GO4320@momjian.us> References: <27596.1571062600@sss.pgh.pa.us> <20191014163538.cqn6j4cl7s2mk2yz@development> <20191015001817.zei45apgpu5dzngr@development> <20191015044118.GO3599@telsasoft.com> <20191015070725.csmdmdvrlb6klwkg@development> <20201206180248.GI24052@telsasoft.com> <20201216172222.GS30237@telsasoft.com> <20201227190729.GB26311@telsasoft.com> <20210112041352.GY1849@telsasoft.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210112041352.GY1849@telsasoft.com> User-Agent: Mutt/1.10.1 (2018-07-13) List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk On Mon, Jan 11, 2021 at 10:13:52PM -0600, Justin Pryzby wrote: > On Mon, Jan 11, 2021 at 03:28:08PM +0100, Peter Eisentraut wrote: > > I think these patches could use some in-place documentation of what they are > > trying to achieve and how they do it. The required information is spread > > over a lengthy thread. No one wants to read that. Add commit messages to > > the patches. > > 0001 patch fixes pg_upgrade/test.sh, which was disfunctional. > Portions of the first patch were independently handled by commits 52202bb39, > fa744697c, 091866724. So this is rebased on those. > I guess updating this script should be a part of a beta-checklist somewhere, > since I guess nobody will want to backpatch changes for testing older releases. Uh, what exactly is missing from the beta checklist? I read the patch and commit message but don't understand it. -- Bruce Momjian https://momjian.us EnterpriseDB https://enterprisedb.com The usefulness of a cup is in its emptiness, Bruce Lee