Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wIiSv-008Pfm-1o for pgsql-hackers@arkaria.postgresql.org; Fri, 01 May 2026 07:39:13 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1wIiSt-00AOaN-2v for pgsql-hackers@arkaria.postgresql.org; Fri, 01 May 2026 07:39:11 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wIiSt-00AOaF-1y for pgsql-hackers@lists.postgresql.org; Fri, 01 May 2026 07:39:11 +0000 Received: from smtp.outgoing.loopia.se ([93.188.3.37]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1wIiSr-00000004DTc-2OYW for pgsql-hackers@lists.postgresql.org; Fri, 01 May 2026 07:39:11 +0000 Received: from s807.loopia.se (localhost [127.0.0.1]) by s807.loopia.se (Postfix) with ESMTP id 948285AD567 for ; Fri, 01 May 2026 09:39:08 +0200 (CEST) Received: from s980.loopia.se (unknown [172.22.191.6]) by s807.loopia.se (Postfix) with ESMTP id 7EBA35AE1C7; Fri, 01 May 2026 09:39:08 +0200 (CEST) Received: from localhost (unknown [172.22.191.6]) by s980.loopia.se (Postfix) with ESMTP id 789102201675; Fri, 01 May 2026 09:39:08 +0200 (CEST) X-Virus-Scanned: amavis at amavis.loopia.se X-Spam-Flag: NO X-Spam-Score: -1.2 X-Spam-Level: X-Spam-Status: No, score=-1.2 tagged_above=-999 required=6.2 tests=[ALL_TRUSTED=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1] autolearn=disabled Authentication-Results: s474.loopia.se (amavis); dkim=pass (2048-bit key) header.d=proxel.se Received: from s980.loopia.se ([172.22.191.5]) by localhost (s474.loopia.se [172.22.190.14]) (amavis, port 10024) with UTF8LMTP id DxfYAr1_xt2w; Fri, 1 May 2026 09:39:08 +0200 (CEST) X-Loopia-Auth: user X-Loopia-User: andreas@proxel.se X-Loopia-Originating-IP: 147.28.75.140 Received: from [192.168.0.186] (customer-147-28-75-140.stosn.net [147.28.75.140]) (Authenticated sender: andreas@proxel.se) by s980.loopia.se (Postfix) with ESMTPSA id 077472201635; Fri, 01 May 2026 09:39:08 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=proxel.se; s=loopiadkim1707418970; t=1777621148; bh=M7Pb2qMXt4wi7HX5bUt4Vlp6bK/S/LRQQMc5Ey1bRcw=; h=Date:Subject:From:To:References:In-Reply-To; b=SWLaCeUu0KzGN/RbbQ1zsU+pVbq0+a1haA/nIUDvk1pVCktEEXXbAvmE3YBjG/kA6 ugDPqbHC2Yn9ZRSDO+GKiGDZ3bKpe1PAHrrzKjvJd9NiVjwkItWm+2C35f/mT0uVbS A6X8zyla0sYy3UBCSlmmgMzwr4QKiADvWZMeUAVpw+W7E0RX748xTX0KlVKsussKcQ xzvJcwKJufnLK4tos7Pvjbc6tJvriJUHJGa8vptnu/tV0yBU458RcfeOJpBqe+mg+s HtF0KKyvxRZbmzBw3U3L7g1BpLTg5hSmK6WDkx9m8wOHK4T18JEtCaSzH/v1oOb6AK K4sTfnHKQzLWg== Message-ID: <0b8303b1-3251-4559-8ab3-5bb5c8271d3b@proxel.se> Date: Fri, 1 May 2026 09:39:07 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] Fix pg_dump emitting OVERRIDING SYSTEM VALUE for tables with dropped identity columns From: Andreas Karlsson To: William Bernbaum , "pgsql-hackers@lists.postgresql.org" References: <49471fc0-5f07-4ed4-b33c-fc634aa301ef@proxel.se> Content-Language: en-US In-Reply-To: <49471fc0-5f07-4ed4-b33c-fc634aa301ef@proxel.se> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 5/1/26 09:28, Andreas Karlsson wrote: > On 4/18/26 01:29, William Bernbaum wrote: >> Patch attached. Forgot this: Maybe it is just a bit too early in the morning and I have not had coffee yet but I struggled a bit to apply your patch. A recommendation for making patches easy to apply for other developers is to generate them with the "git format-patch" command which makes them easy to apply with "git am". You can use the "-v" flag to "git format-patch" if you want to add a version number to the generated file names. Also another question worth looking into is if this same bug affects other things, e.g. the setting of hasdefaults. Thanks again for the patch! -- Andreas Karlsson Percona