Robert Haas escribió:
> I was all prepared to admit that I hadn't actually looked at the patch
> carefully enough, but I just looked at (and CVS HEAD) again and what
> you've written here doesn't appear to describe what I'm seeing in the
> code:
>
> if ((portal->strategy != PORTAL_ONE_SELECT) && (!portal->holdStore))
> FillPortalStore(portal, isTopLevel);
>
> So one of us is confused... it may well be me.
Ah, it seems I misread it ... but then I don't quite see the point in
that change.
Well, not doing a full review anyway, so never mind me.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support