Thread: pgsql: Get rid of some old and crufty global variables in the planner.

pgsql: Get rid of some old and crufty global variables in the planner.

From
tgl@postgresql.org (Tom Lane)
Date:
19 February 2007, 03:03:43
Log Message:
-----------
Get rid of some old and crufty global variables in the planner.  When
this code was last gone over, there wasn't really any alternative to
globals because we didn't have the PlannerInfo struct being passed all
through the planner code.  Now that we do, we can restructure things
to avoid non-reentrancy.  I'm fooling with this because otherwise I'd
have had to add another global variable for the planned compact
range table list.

Modified Files:
--------------
    pgsql/src/backend/nodes:
        outfuncs.c (r1.298 -> r1.299)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c.diff?r1=1.298&r2=1.299)
    pgsql/src/backend/optimizer:
        README (r1.38 -> r1.39)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/README.diff?r1=1.38&r2=1.39)
    pgsql/src/backend/optimizer/path:
        allpaths.c (r1.158 -> r1.159)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/allpaths.c.diff?r1=1.158&r2=1.159)
        clausesel.c (r1.83 -> r1.84)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/clausesel.c.diff?r1=1.83&r2=1.84)
    pgsql/src/backend/optimizer/plan:
        planagg.c (r1.26 -> r1.27)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/planagg.c.diff?r1=1.26&r2=1.27)
        planner.c (r1.212 -> r1.213)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/planner.c.diff?r1=1.212&r2=1.213)
        subselect.c (r1.119 -> r1.120)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/subselect.c.diff?r1=1.119&r2=1.120)
    pgsql/src/backend/optimizer/prep:
        prepjointree.c (r1.46 -> r1.47)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepjointree.c.diff?r1=1.46&r2=1.47)
        preptlist.c (r1.85 -> r1.86)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/preptlist.c.diff?r1=1.85&r2=1.86)
        prepunion.c (r1.137 -> r1.138)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepunion.c.diff?r1=1.137&r2=1.138)
    pgsql/src/backend/optimizer/util:
        clauses.c (r1.234 -> r1.235)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/clauses.c.diff?r1=1.234&r2=1.235)
    pgsql/src/backend/utils/adt:
        selfuncs.c (r1.225 -> r1.226)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c.diff?r1=1.225&r2=1.226)
    pgsql/src/include/nodes:
        nodes.h (r1.194 -> r1.195)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/nodes.h.diff?r1=1.194&r2=1.195)
        primnodes.h (r1.124 -> r1.125)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/primnodes.h.diff?r1=1.124&r2=1.125)
        relation.h (r1.135 -> r1.136)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/relation.h.diff?r1=1.135&r2=1.136)
    pgsql/src/include/optimizer:
        clauses.h (r1.86 -> r1.87)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/clauses.h.diff?r1=1.86&r2=1.87)
        planner.h (r1.36 -> r1.37)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/planner.h.diff?r1=1.36&r2=1.37)
        subselect.h (r1.28 -> r1.29)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/subselect.h.diff?r1=1.28&r2=1.29)