Jeevan Chalke <jeevan.cha...@enterprisedb.com> writes:
> Attached patch.
I would be astonished if this fixes anything. The code still doesn't
know which paths are referenced by which other ones, and so the place
where we free a previously-added path can't know what to do.
I've speculated about adding some form of reference counting to paths
(maybe just a "pin" flag rather than a full refcount) so that we could
be smarter about this. The existing kluge for "don't free IndexPaths"
could be replaced by setting the pin mark on any IndexPath that we
make a bitmap path from. Up to now it hasn't seemed necessary to
generalize that hack, but maybe it's time. Can you show a concrete
case where we are freeing a still-referenced path?
regards, tom lane
