On Fri, Jun 17, 2016 at 4:21 AM, Martín Marqués <martin@2ndquadrant.com> wrote:
> IMO, this assignment:
>
> tbinfo->dobj.dump = tbinfo->dobj.namespace->dobj.dump_contains;
>
> should be replaced by a call to some new function which would be more
> or less a copy of selectDumpableNamespace, but without the
> checkExtensionMembership call.
Ah, I see. Yes this is definitely wrong. The namespace itself may be
part of an extension but we do not check for it at all. See for
example the patch attached that is giving what I would expect is the
correct behavior.
--
Michael