OSDN Git Service

Avoid sharing subpath list structure when flattening nested AppendRels.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 23 Sep 2010 23:34:56 +0000 (19:34 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 23 Sep 2010 19:37:27 +0000 (15:37 -0400)
commitb74ecb9194c9bf35d2775eed08dad777afa40af1
tree959c5eae6c15090bbaf2c13e4e5daf2c884c5196
parent0e6519e11523e8f504d29eb05e4775aa6a847fa5
Avoid sharing subpath list structure when flattening nested AppendRels.

In some situations the original coding led to corrupting the child AppendRel's
subpaths list, effectively adding other members of the parent's list to it.
This was usually masked because we never made any further use of the child's
list, but given the right combination of circumstances, we could do so.  The
visible symptom would be a relation getting scanned twice, as in bug #5673
from David Schmitt.

Backpatch to 8.2, which is as far back as the risky coding appears.  The
example submitted by David only fails in 8.4 and later, but I'm not convinced
that there aren't any even-more-obscure cases where 8.2 and 8.3 would fail.
src/backend/optimizer/path/allpaths.c