Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hIi6X-0001om-Ab for pgsql-hackers@arkaria.postgresql.org; Mon, 22 Apr 2019 23:12:05 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1hIi6W-0007Ky-1J for pgsql-hackers@arkaria.postgresql.org; Mon, 22 Apr 2019 23:12:04 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hIi6V-0007Kq-Ov for pgsql-hackers@lists.postgresql.org; Mon, 22 Apr 2019 23:12:03 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hIi6S-0001Q8-7Y for pgsql-hackers@lists.postgresql.org; Mon, 22 Apr 2019 23:12:02 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id x3MNBsIo031404; Mon, 22 Apr 2019 19:11:54 -0400 From: Tom Lane To: Alvaro Herrera cc: Robert Haas , Amit Langote , PostgreSQL Hackers Subject: Re: pg_dump is broken for partition tablespaces In-reply-to: <20190422225129.GA6126@alvherre.pgsql> References: <20190422225129.GA6126@alvherre.pgsql> Comments: In-reply-to Alvaro Herrera message dated "Mon, 22 Apr 2019 18:51:29 -0400" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <31402.1555974714.1@sss.pgh.pa.us> Date: Mon, 22 Apr 2019 19:11:54 -0400 Message-ID: <31403.1555974714@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Alvaro Herrera writes: > Now that I re-read this complaint once again, I wonder if a mismatching > column order in partitions isn't a thing we ought to preserve anyhow. > Robert, Amit -- is it by design that pg_dump loses the original column > order for partitions, when not in binary-upgrade mode? I haven't looked at the partitioning code, but I am quite sure that that's always happened for old-style inheritance children, and I imagine pg_dump is just duplicating that old behavior. Wasn't there already a patch on the table to change this, though, by removing the code path that uses inheritance rather than the binary-upgrade-like solution? regards, tom lane