]> granicus.if.org Git - postgresql/commit
Take fewer snapshots.
authorRobert Haas <rhaas@postgresql.org>
Wed, 21 Dec 2011 14:16:55 +0000 (09:16 -0500)
committerRobert Haas <rhaas@postgresql.org>
Wed, 21 Dec 2011 14:16:55 +0000 (09:16 -0500)
commitd573e239f03506920938bf0be56c868d9c3416da
tree204b429f7ae848e4413f7df75fc7f98dca0e084e
parente1042a348421bc16f4d4307228a9951e38a984f1
Take fewer snapshots.

When a PORTAL_ONE_SELECT query is executed, we can opportunistically
reuse the parse/plan shot for the execution phase.  This cuts down the
number of snapshots per simple query from 2 to 1 for the simple
protocol, and 3 to 2 for the extended protocol.  Since we are only
reusing a snapshot taken early in the processing of the same protocol
message, the change shouldn't be user-visible, except that the remote
possibility of the planning and execution snapshots being different is
eliminated.

Note that this change does not make it safe to assume that the parse/plan
snapshot will certainly be reused; that will currently only happen if
PortalStart() decides to use the PORTAL_ONE_SELECT strategy.  It might
be worth trying to provide some stronger guarantees here in the future,
but for now we don't.

Patch by me; review by Dimitri Fontaine.
src/backend/commands/portalcmds.c
src/backend/commands/prepare.c
src/backend/executor/spi.c
src/backend/tcop/postgres.c
src/backend/tcop/pquery.c
src/include/tcop/pquery.h