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 1lg76o-0005Vt-RB for pgsql-hackers@arkaria.postgresql.org; Mon, 10 May 2021 14:42:10 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lg76m-0007EA-Vq for pgsql-hackers@arkaria.postgresql.org; Mon, 10 May 2021 14:42:08 +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 1lg76m-0007E2-No for pgsql-hackers@lists.postgresql.org; Mon, 10 May 2021 14:42:08 +0000 Received: from forward4-smtp.messagingengine.com ([66.111.4.238]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lg76i-0008Du-1t for pgsql-hackers@postgresql.org; Mon, 10 May 2021 14:42:07 +0000 Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailforward.nyi.internal (Postfix) with ESMTP id E482E1940A59; Mon, 10 May 2021 10:42:02 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Mon, 10 May 2021 10:42:02 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding: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=fm2; bh=DLsEaDpAFz6JjWKIeL0eAiykIW+mCL/5eXES8vs/L Zc=; b=le3hTkRAeuBkmQV1+E5pg2pW3Wv/XVqMCiOqiCJ0yto7l5J1NUZnNyvrl HP+rieYs1cYEkzCmIyxib1nlBCK2yiyy+HPBBcdVAfxnhhVh0aZM3tASRO0d6Sxo 4HBy2tsaimxbS/vWOyJd7MTRtcCztrz3NgCjh7aEiINcNCp/sxVE9dCOtstDDnbJ JGnbJNMRhM2Ok2DBxAMD4artlzQn1S1J7Va8z8GxD74lOWQPgLKxaIUOPi3MOjVm HkEwF/cHEwGcc6yTUmltQ6vO5Z0a+ZlCtEzd6MbmdlODeoAF8UCddEGR/N4B827j KHgklegiug9Cou4XSwUhqO9q6alXQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrvdegkedgkedtucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepuffvfhfhkffffgggjggtgfesthejredttdefjeenucfhrhhomheprfgvthgv rhcugfhishgvnhhtrhgruhhtuceophgvthgvrhdrvghishgvnhhtrhgruhhtsegvnhhtvg hrphhrihhsvggusgdrtghomheqnecuggftrfgrthhtvghrnhepleevheehueeiudeiteei tdekueetleefteejffeiffdugedvffdvffdtiedvueeknecukfhppeeluddrudegrddvfe eirdeivdenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhm pehpvghtvghrrdgvihhsvghnthhrrghuthesvghnthgvrhhprhhishgvuggsrdgtohhm X-ME-Proxy: Received: from laptop206oxuk.local (p5b0eec3e.dip0.t-ipconnect.de [91.14.236.62]) by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 10 May 2021 10:42:01 -0400 (EDT) Subject: Re: SQL-standard function body To: Tom Lane Cc: Justin Pryzby , Noah Misch , Robert Haas , pgsql-hackers@postgresql.org References: <1c11f1eb-f00c-43b7-799d-2d44132c02d7@2ndquadrant.com> <493812.1593541480@sss.pgh.pa.us> <494838.1593543098@sss.pgh.pa.us> <20210418185546.GC1500781@rfd.leadboat.com> <92408.1618772924@sss.pgh.pa.us> <20210418211505.GA7256@telsasoft.com> <142618.1618781618@sss.pgh.pa.us> <964466e9-6b6c-c0ae-6277-a44e80229299@enterprisedb.com> <2659727.1619540213@sss.pgh.pa.us> From: Peter Eisentraut Message-ID: Date: Mon, 10 May 2021 16:41:58 +0200 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 MIME-Version: 1.0 In-Reply-To: <2659727.1619540213@sss.pgh.pa.us> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 27.04.21 18:16, Tom Lane wrote: > That's kind of a lot of complication, and inefficiency, for a corner case > that may never arise in practice. We've ignored the risk for default > expressions, and AFAIR have yet to receive any field complaints about it. > So maybe it's okay to do the same for SQL-style function bodies, at least > for now. > >> Another option would be that we disallow this at creation time. > > Don't like that one much. The backend shouldn't be in the business > of rejecting valid commands just because pg_dump might be unable > to cope later. Since this is listed as an open item, I want to clarify that I'm currently not planning to work on this, based on this discussion. Certainly something to look into sometime later, but it's not in my plans right now.