From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-patches(at)postgresql(dot)org |
Subject: | Re: Move cursor support for pl/pgsql |
Date: | 2007-03-02 15:49:24 |
Message-ID: | 45E84784.2030006@hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Tom Lane wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
>> I took a stab at implementing MOVE support for cursors in PL/pgsql,
>> because I wanted it for a project. Attached patch *seems* to work. But..
>> Given that this is my first venture into touching bison/flex files, I
>> may be completely off-target in how it's done.
>
> I disapprove of hard-wiring the fetch count as an integer constant;
> there's no good reason not to treat it as an expression. So you should
> drop the T_NUMBER thing.
Will do, thanks for commenting. I'm also going to merge it with Pavels
work that I had completely missed earlier.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Gregory Stark | 2007-03-02 16:25:53 | Re: A little COPY speedup |
Previous Message | Tom Lane | 2007-03-02 15:17:14 | Re: Move cursor support for pl/pgsql |