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 1l23fP-00037z-A3 for pgsql-general@arkaria.postgresql.org; Wed, 20 Jan 2021 02:56:20 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1l23fN-0002pY-6k for pgsql-general@arkaria.postgresql.org; Wed, 20 Jan 2021 02:56:17 +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 1l23fL-0002pQ-Ib for pgsql-general@lists.postgresql.org; Wed, 20 Jan 2021 02:56:16 +0000 Received: from wout2-smtp.messagingengine.com ([64.147.123.25]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1l23fH-0000Gj-Kq for pgsql-general@postgresql.org; Wed, 20 Jan 2021 02:56:14 +0000 Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.west.internal (Postfix) with ESMTP id 9C22816BA; Tue, 19 Jan 2021 21:56:09 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Tue, 19 Jan 2021 21:56:09 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=paquier.xyz; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm3; bh=dc6lFFgArTaMvTRQ7xnebBFsh8p LJWY7xtTWC9tKXuM=; b=ub1uPt3L1vLyYxu715q4pU1Z5YAT9b8uleRtoFOgvJ/ ChEblQgOpUf+ytkDKqPNxQSFtNMphluwMn939b/+Aep5yRxYDR36AQBQ/j6aidgj nkd8jTtyDjFt9xSo4MmSphN6NmoM2qm1sfCj70BltRkhPHWvzZ3owi8cH68rRKTG r7VG38/TAcnxtkEVvvTluydD5J5LOLBlttOYqnS0yG+rDtxm2b2aAW0h44aW2QA5 LLvyvwYZznNGi78R3ieZQPCGMejR/VSnLC7IjW8W4rmfZ2UGXA0Cx5zlLJ1TGy8P pWXM1XHNTTgqv+zR4i53xEAabfn8L4tMx/4UpKO83ow== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=dc6lFF gArTaMvTRQ7xnebBFsh8pLJWY7xtTWC9tKXuM=; b=MqcUzP/dL5nCJTUB7Mlbr1 /g5jY0Evy3hyE/zJpDSHxZj0T+M+wxJH4mT1beQJkejlPE+7eNV/HpIct6rUFF6b cMpM3K+AsJebdipKVC5cGfXY5o+I5mU1ehZC7bCxyTQOZQRwOdI+1drsSz+iwpcU mm+PzOxVD/J7LNRYI9rrQAu3bRoM0B7vO6XtXn9z7E8j7Ay79z0IOrUMD+wAO3yE rpAQbVSGcOfEcPU3q/Zlz1hsQPi2Nr8NMi8HurUPsBW22/QFdPW7/meHh+QrsaaB DRprkxGUyAXZoAjijPNMOJcSFP2Jey/ez/kY7kNa+hA+rOoulSX+wbzr5daTfVdw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledruddugdehgecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenfg hrlhcuvffnffculdejtddmnecujfgurhepfffhvffukfhfgggtuggjsehgtderredttddv necuhfhrohhmpefoihgthhgrvghlucfrrghquhhivghruceomhhitghhrggvlhesphgrqh huihgvrhdrgiihiieqnecuggftrfgrthhtvghrnhepvdegudeuhfdtueeltedtveejheeh ieevueeigeelteegleejleeiueeiheegvefhnecukfhppeduuddurddutddvrddukedtrd dukeehnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhep mhhitghhrggvlhesphgrqhhuihgvrhdrgiihii X-ME-Proxy: Received: from paquier.xyz (ee0822lan1.rev.em-net.ne.jp [111.102.180.185]) by mail.messagingengine.com (Postfix) with ESMTPA id 7301A1080057; Tue, 19 Jan 2021 21:56:07 -0500 (EST) Date: Wed, 20 Jan 2021 11:56:03 +0900 From: Michael Paquier To: Alvaro Herrera Cc: Andrus , pgsql-general@postgresql.org Subject: Re: Error messages on duplicate schema names Message-ID: References: <20210119203751.GA5473@alvherre.pgsql> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="jtN5YEHWOJTQclLR" Content-Disposition: inline In-Reply-To: <20210119203751.GA5473@alvherre.pgsql> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --jtN5YEHWOJTQclLR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Jan 19, 2021 at 05:37:51PM -0300, Alvaro Herrera wrote: > I guess you could make the case that the CCI call should be in the > callers where we actually loop (SetDefaultACLsInSchemas, > RemoveRoleFromObjectACL), but it's hard to get excited about the added > clutter. Yeah, that matches my thoughts. And I did not see any harm in putting the CCI in SetDefaultACL() even for the case of pg_default_acl with DROP OWNED BY. So applied and backpatched. Thanks, Alvaro and Andrus. -- Michael --jtN5YEHWOJTQclLR Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmAHm8MACgkQnvQgOdby QH1AYA/9HBTdKdJaXIiTy63i7gDtd5VwlU/uAb6sAgxNzlM1s9/Rc7XzM1OoqD1g eZJlvZHn2kzCjuN0Krb8tHOkZ0UAKl+DzwM12N1QVBoigDCHTIMsNJyoXbM0wm6s 9jJL7pAVZxC0Oxouq4az0i84DF3QmQ6Asf/Zf+opl/nE7G1jvOyHT921XnLsMPuA OlRgZKMOuBCVRcG93RSl50Iz3j/Bq6vv5QIfwget7y/npp7QdjwAURNAWEmHc0D+ tlC0xsw/0wFaNZhYFHbKMMbL96DKge+qDr+IMCEMcgkKh3+ghzHnmtRxAEWIG+7j XP6bV/K4vKtUl8VJhCzWVTjXheyXncartAaNcRKhqRh8msDljkzOAVcJERLFqe5u CzLe6m8//FBB0/Qn97eXVAfinmnlXjoRm5D5CD1oOrjc6AIai/p/kRdgeUQ5TU73 vO8MieiOBUztT5gKquZFgv+/SP077QrmR4xoQ13/yDg23TqPAB+1jvzvpLhU0SDk V/DhcttIR/mchaLTyImpNoVObJsOc9hDZorE3jOXgVEvCQSW3AZ/c+l0qIQ7+ga2 KKk1oiVs5meSSw03vAILWKAdkn9Nqql8pc3dpOMNy+mXdOAmfZM1ypYKKswSkaOL uSucQChWHAIrWuVWNPTpf4sIyoTFgXIJWZ4d11ai2H80XcrOjr0= =vtZm -----END PGP SIGNATURE----- --jtN5YEHWOJTQclLR--