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 1nRtfc-00089i-6r for pgsql-hackers@arkaria.postgresql.org; Wed, 09 Mar 2022 10:35:52 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1nRtfa-00087P-E6 for pgsql-hackers@arkaria.postgresql.org; Wed, 09 Mar 2022 10:35:50 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nRtfZ-00086o-3H for pgsql-hackers@lists.postgresql.org; Wed, 09 Mar 2022 10:35:50 +0000 Received: from wout1-smtp.messagingengine.com ([64.147.123.24]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nRtfP-0002iG-Nc for pgsql-hackers@postgresql.org; Wed, 09 Mar 2022 10:35:48 +0000 Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id BC116320093A; Wed, 9 Mar 2022 05:35:35 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Wed, 09 Mar 2022 05:35:36 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=yGaRLJrAV87RELIdzkoJEWf/9eqx1kdZ7W6cGm5SX Gw=; b=hwnuE7cMgoLXJcO7XMkZgW6vDN32q24z4tRiNDQSm+4qreLwLQb42vn0y Ftzh7zKcgmiyiitx3ii1BuivaScb53/F1h7yN7TYUm9FsxAAdybawH5JyNjm/kz6 PT0OhHnqMuYl98Tmo1B72Ipe0CPzUgMR8YYgQFZrMV3pWF0qAQDPRL+yESX/JhSR liwMl+CzmBoMNMYdiWB4MILeOHnqoku2D4hGZoynoLohx1I2Y0FDarcx+JRSjs1p n9fQvRwYPP1GACr+z+S3QMw7h1kIZOoZc8lTteVxQhHKm9f4P0wQdgtBWnMYaDSL vj4UndHIjX7roa2pT+EHyurAVkLEA== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvvddruddukedgudejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvfhfhjggtgfesthejredttdefjeenucfhrhhomheprfgvthgv rhcugfhishgvnhhtrhgruhhtuceophgvthgvrhdrvghishgvnhhtrhgruhhtsegvnhhtvg hrphhrihhsvggusgdrtghomheqnecuggftrfgrthhtvghrnhepfeejgeehteeuhfevvedu leeufedtjeetiefftedvudfhtdeifefgueettdevgefgnecuvehluhhsthgvrhfuihiivg eptdenucfrrghrrghmpehmrghilhhfrhhomhepphgvthgvrhdrvghishgvnhhtrhgruhht segvnhhtvghrphhrihhsvggusgdrtghomh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 9 Mar 2022 05:35:34 -0500 (EST) Message-ID: <71b7dd2a-a47d-94a7-d677-961351dac85e@enterprisedb.com> Date: Wed, 9 Mar 2022 11:35:32 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.6.2 Subject: Re: parse/analyze API refactoring Content-Language: en-US To: Nathan Bossart Cc: "Bossart, Nathan" , pgsql-hackers References: <72519c4f-ebd8-b796-8216-1e6213fc1cf1@enterprisedb.com> <20220228185121.GA944837@nathanxps13> From: Peter Eisentraut In-Reply-To: <20220228185121.GA944837@nathanxps13> 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 28.02.22 19:51, Nathan Bossart wrote: > On Mon, Feb 28, 2022 at 07:46:40AM +0100, Peter Eisentraut wrote: >> You set this commit fest entry to Waiting on Author, but there were no >> reviews posted and the patch still applies and builds AFAICT, so I don't >> know what you meant by that. > > Apologies for the lack of clarity. I believe my only feedback was around > deduplicating the pg_analyze_and_rewrite_*() functions. Would you rather > handle that in a separate patch? I have committed my original patches. I'll leave the above-mentioned topic as ideas for the future.