pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: David Rowley <drowley@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Update wording in optimizer/README for EquivalenceClasses
Date: Thu, 10 Apr 2025 05:34:56 +0000
Message-ID: <E1u2kYy-003YDw-2R@gemulon.postgresql.org> (raw)

Update wording in optimizer/README for EquivalenceClasses

d69d45a5a changed how em_is_child members are stored in
EquivalenceClasses.  Children are no longer stored in the ec_members
list.  optimizer/README mentioned that most operations "should ignore
child members", but that felt a little untrue now since child members
are now stored in a separate place, they simply won't be found by the
normal means of looking (a foreach loop over ec_members), and if you don't
find them, there's technically no need to "ignore" them.

Here we tweak the wording slightly to reflect the new storage location
for child members.

Reported-by: Amit Langote <amitlangote09@gmail.com>
Author: Amit Langote <amitlangote09@gmail.com>
Author: David Rowley <dgrowleyml@gmail.com>
Discussion: https://postgr.es/m/CA+HiwqE8v=EuAP_3F_A2xn8zWx+nG_etW_Fe_DvKO-Fkx=+DdQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d7c04db27aeba1718d8059eae9a6e6a5eedc01b8

Modified Files
--------------
src/backend/optimizer/README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)



Message-ID: <E1u2kYy-003YDw-2R@gemulon.postgresql.org>
Permalink:  ../E1u2kYy-003YDw-2R@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1u2kYy-003YDw-2R@gemulon.postgresql.org

 · 

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-committers@postgresql.org
  Cc: drowley@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Update wording in optimizer/README for EquivalenceClasses
  In-Reply-To: <E1u2kYy-003YDw-2R@gemulon.postgresql.org>

* 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