Re: [PATCH] Provide rowcount for utility SELECTs
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Alvaro Herrera <alvherre@commandprompt.com>
Cc: Boszormenyi Zoltan <zb@cybertec.at>, Tom Lane <tgl@sss.pgh.pa.us>, Peter Eisentraut <peter_e@gmx.net>, Hans-Juergen Schoenig <hs@cybertec.at>, Pavel Stehule <pavel.stehule@gmail.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2010-02-11T19:50:42Z
Lists: pgsql-hackers
On Thu, Feb 11, 2010 at 2:47 PM, Alvaro Herrera <alvherre@commandprompt.com> wrote: > Robert Haas escribió: >> On Thu, Feb 11, 2010 at 2:38 PM, Alvaro Herrera >> <alvherre@commandprompt.com> wrote: >> > 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 the point is just that Zoltan is adding some more code that >> applies to both branches of the switch, so merging them saves some >> duplication. > > But then there's no other branches, so why not just put it below the > switch? No, PORTAL_MULTI_QUERY is still separate. >> > Well, not doing a full review anyway, so never mind me. >> >> Actually I was sort of hoping you (or someone other than me) would >> pick this up for commit... > > Hmm ... Maybe I came to the wrong place. :-) ...Robert