Talk About Network

Google


Register and Login
Nick
Password
Register create new account Sign up is FREE and you can post replies, new topics, bookmark posts and more!
Recover lost password


Data Bases > Pgsql Committers > pgsql: As noted...
Latest [ Topics | Posts ] Archive Post A New Topic Post a Reply
<< Topic < Post Post 1 of 1 Topic 16954 of 18199
Post > Topic >>

pgsql: As noted by Andrew Gierth, there's really no need any more to

by tgl@[EMAIL PROTECTED] (Tom Lane) Jul 26, 2008 at 07:15 PM

Log Message:
-----------
As noted by Andrew Gierth, there's really no need any more to force a junk
filter to be used when INSERT or SELECT INTO has a plan that returns raw
disk tuples.  The virtual-tuple-slot optimizations that were put in place
awhile ago mean that ExecInsert has to do ExecMaterializeSlot, and that
already copies the tuple if it's raw (and does so more efficiently than
a junk filter, too).  So get rid of that logic.  This in turn means that
we can throw away ExecMayReturnRawTuples, which wasn't used for any other
purpose, and was always a kluge anyway.

In passing, move a couple of SELECT-INTO-specific fields out of EState
and into the private state of the SELECT INTO DestReceiver, as was
foreseen
in an old comment there.  Also make intorel_receive use
ExecMaterializeSlot
not ExecCopySlotTuple, for consistency with ExecInsert and to possibly
save
a tuple copy step in some cases.

Modified Files:
--------------
    pgsql/src/backend/executor:
        execAmi.c (r1.95 -> r1.96)
       
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execAmi.c?r1=1.95&r2=1.96)
        execMain.c (r1.310 -> r1.311)
       
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execMain.c?r1=1.310&r2=1.311)
        execUtils.c (r1.155 -> r1.156)
       
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execUtils.c?r1=1.155&r2=1.156)
    pgsql/src/include/executor:
        executor.h (r1.148 -> r1.149)
       
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/executor/executor.h?r1=1.148&r2=1.149)
    pgsql/src/include/nodes:
        execnodes.h (r1.184 -> r1.185)
       
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/execnodes.h?r1=1.184&r2=1.185)

-- 
Sent via pgsql-committers mailing list (pgsql-committers@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers
 




 1 Posts in Topic:
pgsql: As noted by Andrew Gierth, there's really no need any mor
tgl@[EMAIL PROTECTED] (T  2008-07-26 19:15:35 

Post A Reply:
  Go here to Signup

AddThis Feed Button


About - Advertising - Contact - Frequently Asked Questions - Privacy Policy - Terms of Use - Signup

Contact
tan12V112 Fri Dec 5 6:40:27 CST 2008.