pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-hackers@lists.postgresql.org
Subject: Re: Convert planner's AggInfo and AggTransInfo to Nodes
Date: Tue, 19 Jul 2022 17:49:34 +0200
Message-ID: <3f5d4811-92db-a358-0ef4-c69adf974d61@enterprisedb.com> (raw)
In-Reply-To: <742479.1658160504@sss.pgh.pa.us>
References: <742479.1658160504@sss.pgh.pa.us>

On 18.07.22 18:08, Tom Lane wrote:
> I'm kind of tempted to mount an effort to get rid of as many of
> pathnodes.h's "read_write_ignore" annotations as possible.  Some are
> necessary to prevent infinite recursion, and others represent considered
> judgments that they'd bloat node dumps more than they're worth --- but
> I think quite a lot of them arose from plain laziness about updating
> outfuncs.c.  With the infrastructure we have now, that's no longer
> a good reason.

That was my impression as well, and I agree it would be good to sort 
that out.





view thread (6+ messages)  latest in thread

Message-ID: <3f5d4811-92db-a358-0ef4-c69adf974d61@enterprisedb.com>
Permalink:  ../3f5d4811-92db-a358-0ef4-c69adf974d61@enterprisedb.com/
Also on:    postgresql.org/message-id/3f5d4811-92db-a358-0ef4-c69adf974d61@enterprisedb.com

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: peter.eisentraut@enterprisedb.com, tgl@sss.pgh.pa.us, pgsql-hackers@lists.postgresql.org
  Subject: Re: Convert planner's AggInfo and AggTransInfo to Nodes
  In-Reply-To: <3f5d4811-92db-a358-0ef4-c69adf974d61@enterprisedb.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox