Avoid sharing subpath list structure when flattening nested AppendRels.
authorTom Lane <[email protected]>
Thu, 23 Sep 2010 23:34:56 +0000 (19:34 -0400)
committerTom Lane <[email protected]>
Thu, 23 Sep 2010 19:41:51 +0000 (19:41 +0000)
commit9e20bdf9a8417615f40ddf3c8e6e34be2e7e88e2
tree55b5a93aa8995da9ecdc485a41e5d9a284d4315d
parent33f41109fa706d3e445ae9365aaab97f1886d88c
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