pg.ddx.io pgsql-committers@postgresql.org mailing list archive
help / color / mirror / Atom feedpgsql: Fix set_append_rel_pathlist() to deal intelligently with cases
3+ messages / 2 participants
[nested] [flat]
* pgsql: Fix set_append_rel_pathlist() to deal intelligently with cases
@ 2009-07-06 18:26 Tom Lane <tgl@postgresql.org>
0 siblings, 0 replies; 3+ messages in thread
From: tgl@postgresql.org @ 2009-07-06 18:26 UTC (permalink / raw)
To: pgsql-committers
Log Message:
-----------
Fix set_append_rel_pathlist() to deal intelligently with cases where
substituting a child rel's output expressions into the appendrel's restriction
clauses yields a pseudoconstant restriction. We might be able to skip scanning
that child rel entirely (if we get constant FALSE), or generate a one-time
filter. 8.3 more or less accidentally generated plans that weren't completely
stupid in these cases, but that was only because an extra recursive level of
subquery_planner() always occurred and allowed const-simplification to happen.
8.4's ability to pull up appendrel members with non-Var outputs exposes the
fact that we need to work harder here. Per gripe from Sergey Burladyan.
Modified Files:
--------------
pgsql/src/backend/optimizer/path:
allpaths.c (r1.183 -> r1.184)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/allpaths.c?r1=1.183&r2...)
pgsql/src/backend/optimizer/prep:
prepunion.c (r1.171 -> r1.172)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepunion.c?r1=1.171&r...)
pgsql/src/backend/optimizer/util:
restrictinfo.c (r1.60 -> r1.61)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/restrictinfo.c?r1=1.60&...)
pgsql/src/include/optimizer:
restrictinfo.h (r1.44 -> r1.45)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/restrictinfo.h?r1=1.44&r2=1...)
^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: Fix set_append_rel_pathlist() to deal intelligently with cases
@ 2009-07-06 18:26 Tom Lane <tgl@postgresql.org>
0 siblings, 0 replies; 3+ messages in thread
From: tgl@postgresql.org @ 2009-07-06 18:26 UTC (permalink / raw)
To: pgsql-committers
Log Message:
-----------
Fix set_append_rel_pathlist() to deal intelligently with cases where
substituting a child rel's output expressions into the appendrel's restriction
clauses yields a pseudoconstant restriction. We might be able to skip scanning
that child rel entirely (if we get constant FALSE), or generate a one-time
filter. 8.3 more or less accidentally generated plans that weren't completely
stupid in these cases, but that was only because an extra recursive level of
subquery_planner() always occurred and allowed const-simplification to happen.
8.4's ability to pull up appendrel members with non-Var outputs exposes the
fact that we need to work harder here. Per gripe from Sergey Burladyan.
Tags:
----
REL8_4_STABLE
Modified Files:
--------------
pgsql/src/backend/optimizer/path:
allpaths.c (r1.183 -> r1.183.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/allpaths.c?r1=1.183&r2...)
pgsql/src/backend/optimizer/prep:
prepunion.c (r1.171 -> r1.171.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepunion.c?r1=1.171&r...)
pgsql/src/backend/optimizer/util:
restrictinfo.c (r1.60 -> r1.60.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/restrictinfo.c?r1=1.60&...)
pgsql/src/include/optimizer:
restrictinfo.h (r1.44 -> r1.44.2.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/restrictinfo.h?r1=1.44&r2=1...)
^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: Pass CAC as an argument to the backend process
@ 2024-03-12 11:45 Heikki Linnakangas <heikki.linnakangas@iki.fi>
0 siblings, 0 replies; 3+ messages in thread
From: Heikki Linnakangas @ 2024-03-12 11:45 UTC (permalink / raw)
To: pgsql-committers@lists.postgresql.org
Pass CAC as an argument to the backend process
We used to smuggle it to the child process in the Port struct, but it
seems better to pass it down as a separate argument. This paves the
way for the next commit, which moves the initialization of the Port
struct to the backend process, after forking.
Reviewed-by: Tristan Partin, Andres Freund
Discussion: https://www.postgresql.org/message-id/7a59b073-5b5b-151e-7ed3-8b01ff7ce9ef@iki.fi
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/d162c3a73bf14416ff4012de6f01c3d825610f70
Modified Files
--------------
src/backend/postmaster/postmaster.c | 44 +++++++++++++++++++++++++++----------
src/include/libpq/libpq-be.h | 12 ----------
2 files changed, 32 insertions(+), 24 deletions(-)
^ permalink raw reply [nested|flat] 3+ messages in thread
end of thread, other threads:[~2024-03-12 11:45 UTC | newest]
Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2009-07-06 18:26 pgsql: Fix set_append_rel_pathlist() to deal intelligently with cases Tom Lane <tgl@postgresql.org>
2009-07-06 18:26 pgsql: Fix set_append_rel_pathlist() to deal intelligently with cases Tom Lane <tgl@postgresql.org>
2024-03-12 11:45 pgsql: Pass CAC as an argument to the backend process Heikki Linnakangas <heikki.linnakangas@iki.fi>
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