pgsql: Cosmetic improvements in setup of planner's per-RTE arrays.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Cosmetic improvements in setup of planner's per-RTE arrays.
Date: 2019-08-09 16:33:49
Message-ID: E1hw7pt-0006rv-Li@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Cosmetic improvements in setup of planner's per-RTE arrays.

Merge setup_append_rel_array into setup_simple_rel_arrays. There's no
particularly good reason to keep them separate, and it's inconsistent
with the lack of separation in expand_planner_arrays. The only apparent
benefit was that the fast path for trivial queries in query_planner()
doesn't need to set up the append_rel_array; but all we're saving there
is an if-test and NULL assignment, which surely ought to be negligible.

Also improve some obsolete comments.

Discussion: https://postgr.es/m/17220.1565301350@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1661a4050593a472c369a6660ffec05b6b837c57

Modified Files
--------------
src/backend/optimizer/plan/planmain.c | 10 +------
src/backend/optimizer/prep/prepunion.c | 8 +----
src/backend/optimizer/util/relnode.c | 53 ++++++++++++++++++----------------
src/include/nodes/pathnodes.h | 9 +++---
src/include/optimizer/pathnode.h | 1 -
5 files changed, 34 insertions(+), 47 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-08-09 17:20:43 pgsql: Fix SIGSEGV in pruning for ScalarArrayOp with constant-null arra
Previous Message Michael Paquier 2019-08-09 02:12:16 pgsql: Refactor logic to remove trailing CR/LF characters from strings