Skip site navigation (1) Skip section navigation (2)

pgsql: Change the planner-to-executor API so that the planner tells the

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Change the planner-to-executor API so that the planner tells the
Date: 2007-01-10 18:06:05
Message-ID: 20070110180605.1E7899FA3A3@postgresql.org (view raw or flat)
Thread:
Lists: pgsql-committers
Log Message:
-----------
Change the planner-to-executor API so that the planner tells the executor
which comparison operators to use for plan nodes involving tuple comparison
(Agg, Group, Unique, SetOp).  Formerly the executor looked up the default
equality operator for the datatype, which was really pretty shaky, since it's
possible that the data being fed to the node is sorted according to some
nondefault operator class that could have an incompatible idea of equality.
The planner knows what it has sorted by and therefore can provide the right
equality operator to use.  Also, this change moves a couple of catalog lookups
out of the executor and into the planner, which should help startup time for
pre-planned queries by some small amount.  Modify the planner to remove some
other cavalier assumptions about always being able to use the default
operators.  Also add "nulls first/last" info to the Plan node for a mergejoin
--- neither the executor nor the planner can cope yet, but at least the API is
in place.

Modified Files:
--------------
    pgsql/src/backend/executor:
        execGrouping.c (r1.22 -> r1.23)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execGrouping.c.diff?r1=1.22&r2=1.23)
        nodeAgg.c (r1.148 -> r1.149)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeAgg.c.diff?r1=1.148&r2=1.149)
        nodeGroup.c (r1.66 -> r1.67)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeGroup.c.diff?r1=1.66&r2=1.67)
        nodeMergejoin.c (r1.84 -> r1.85)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeMergejoin.c.diff?r1=1.84&r2=1.85)
        nodeSetOp.c (r1.23 -> r1.24)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeSetOp.c.diff?r1=1.23&r2=1.24)
        nodeUnique.c (r1.54 -> r1.55)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeUnique.c.diff?r1=1.54&r2=1.55)
    pgsql/src/backend/nodes:
        copyfuncs.c (r1.360 -> r1.361)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/copyfuncs.c.diff?r1=1.360&r2=1.361)
        equalfuncs.c (r1.294 -> r1.295)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/equalfuncs.c.diff?r1=1.294&r2=1.295)
        outfuncs.c (r1.292 -> r1.293)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c.diff?r1=1.292&r2=1.293)
    pgsql/src/backend/optimizer/path:
        costsize.c (r1.173 -> r1.174)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/costsize.c.diff?r1=1.173&r2=1.174)
        joinpath.c (r1.109 -> r1.110)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/joinpath.c.diff?r1=1.109&r2=1.110)
    pgsql/src/backend/optimizer/plan:
        createplan.c (r1.220 -> r1.221)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/createplan.c.diff?r1=1.220&r2=1.221)
        planner.c (r1.210 -> r1.211)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/planner.c.diff?r1=1.210&r2=1.211)
        subselect.c (r1.116 -> r1.117)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/subselect.c.diff?r1=1.116&r2=1.117)
    pgsql/src/backend/optimizer/util:
        clauses.c (r1.228 -> r1.229)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/clauses.c.diff?r1=1.228&r2=1.229)
        pathnode.c (r1.135 -> r1.136)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/pathnode.c.diff?r1=1.135&r2=1.136)
    pgsql/src/backend/parser:
        parse_clause.c (r1.162 -> r1.163)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/parse_clause.c.diff?r1=1.162&r2=1.163)
    pgsql/src/backend/utils/cache:
        lsyscache.c (r1.142 -> r1.143)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/cache/lsyscache.c.diff?r1=1.142&r2=1.143)
    pgsql/src/backend/utils/sort:
        tuplesort.c (r1.73 -> r1.74)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/sort/tuplesort.c.diff?r1=1.73&r2=1.74)
    pgsql/src/include/executor:
        executor.h (r1.133 -> r1.134)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/executor/executor.h.diff?r1=1.133&r2=1.134)
    pgsql/src/include/nodes:
        plannodes.h (r1.88 -> r1.89)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/plannodes.h.diff?r1=1.88&r2=1.89)
        relation.h (r1.131 -> r1.132)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/relation.h.diff?r1=1.131&r2=1.132)
    pgsql/src/include/optimizer:
        pathnode.h (r1.74 -> r1.75)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/pathnode.h.diff?r1=1.74&r2=1.75)
        planmain.h (r1.96 -> r1.97)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/planmain.h.diff?r1=1.96&r2=1.97)
    pgsql/src/include/utils:
        lsyscache.h (r1.111 -> r1.112)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/lsyscache.h.diff?r1=1.111&r2=1.112)

pgsql-committers by date

Next:From: Tom LaneDate: 2007-01-10 18:22:50
Subject: pgsql: Make sure BYTE_ORDER gets defined in 64-bit builds on Solaris,
Previous:From: Tom LaneDate: 2007-01-10 14:17:19
Subject: Re: pgsql: Change Windows rename and unlink substitutes so that they time

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group