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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers