Обсуждение: todo point: plpgsql - scrollable cursors are supported

Поиск
Список
Период
Сортировка

todo point: plpgsql - scrollable cursors are supported

От
Pavel Stehule
Дата:
Hello

I am thinking so you can remove a "scrollable cursor support" from
ToDo for plpgsql. Scrollable cursors are supported and supported
syntax are same as core SQL language.

regards

Pavel Stehule


Re: todo point: plpgsql - scrollable cursors are supported

От
Robert Haas
Дата:
On Mon, Oct 4, 2010 at 2:52 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> I am thinking so you can remove a "scrollable cursor support" from
> ToDo for plpgsql. Scrollable cursors are supported and supported
> syntax are same as core SQL language.

I agree, removed.  I also removed WITH HOLD cursors, which we seem to
have as well.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


Re: todo point: plpgsql - scrollable cursors are supported

От
Pavel Stehule
Дата:
2010/10/7 Robert Haas <robertmhaas@gmail.com>:
> On Mon, Oct 4, 2010 at 2:52 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>> I am thinking so you can remove a "scrollable cursor support" from
>> ToDo for plpgsql. Scrollable cursors are supported and supported
>> syntax are same as core SQL language.
>
> I agree, removed.  I also removed WITH HOLD cursors, which we seem to
> have as well.

I think so we doesn't support WITH HOLD cursor syntax yet. Maybe we
have similar functionality. Don't know.

Pavel

>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise Postgres Company
>


Re: todo point: plpgsql - scrollable cursors are supported

От
Robert Haas
Дата:
On Thu, Oct 7, 2010 at 2:38 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> 2010/10/7 Robert Haas <robertmhaas@gmail.com>:
>> On Mon, Oct 4, 2010 at 2:52 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>>> I am thinking so you can remove a "scrollable cursor support" from
>>> ToDo for plpgsql. Scrollable cursors are supported and supported
>>> syntax are same as core SQL language.
>>
>> I agree, removed.  I also removed WITH HOLD cursors, which we seem to
>> have as well.
>
> I think so we doesn't support WITH HOLD cursor syntax yet. Maybe we
> have similar functionality. Don't know.

It's in the documentation...

http://www.postgresql.org/docs/current/static/sql-declare.html

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


Re: todo point: plpgsql - scrollable cursors are supported

От
Pavel Stehule
Дата:
2010/10/7 Robert Haas <robertmhaas@gmail.com>:
> On Thu, Oct 7, 2010 at 2:38 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>> 2010/10/7 Robert Haas <robertmhaas@gmail.com>:
>>> On Mon, Oct 4, 2010 at 2:52 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>>>> I am thinking so you can remove a "scrollable cursor support" from
>>>> ToDo for plpgsql. Scrollable cursors are supported and supported
>>>> syntax are same as core SQL language.
>>>
>>> I agree, removed.  I also removed WITH HOLD cursors, which we seem to
>>> have as well.
>>
>> I think so we doesn't support WITH HOLD cursor syntax yet. Maybe we
>> have similar functionality. Don't know.
>
> It's in the documentation...
>
> http://www.postgresql.org/docs/current/static/sql-declare.html

can be - but it isn't supported by plpgsql. This is similar to
scrollable cursors - SQL supported it some years, but plpgsql allowed
it only two years. The minimal necessary change is relative simple
plpgsql's parser update - but I don't know if there isn't other
issues.

see http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/gram.y.diff?r1=1.99&r2=1.100

Regards

Pavel


>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise Postgres Company
>


Re: todo point: plpgsql - scrollable cursors are supported

От
Robert Haas
Дата:
On Thu, Oct 7, 2010 at 7:34 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> 2010/10/7 Robert Haas <robertmhaas@gmail.com>:
>> On Thu, Oct 7, 2010 at 2:38 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>>> 2010/10/7 Robert Haas <robertmhaas@gmail.com>:
>>>> On Mon, Oct 4, 2010 at 2:52 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>>>>> I am thinking so you can remove a "scrollable cursor support" from
>>>>> ToDo for plpgsql. Scrollable cursors are supported and supported
>>>>> syntax are same as core SQL language.
>>>>
>>>> I agree, removed.  I also removed WITH HOLD cursors, which we seem to
>>>> have as well.
>>>
>>> I think so we doesn't support WITH HOLD cursor syntax yet. Maybe we
>>> have similar functionality. Don't know.
>>
>> It's in the documentation...
>>
>> http://www.postgresql.org/docs/current/static/sql-declare.html
>
> can be - but it isn't supported by plpgsql. This is similar to
> scrollable cursors - SQL supported it some years, but plpgsql allowed
> it only two years. The minimal necessary change is relative simple
> plpgsql's parser update - but I don't know if there isn't other
> issues.
>
> see http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/gram.y.diff?r1=1.99&r2=1.100

I think the issue there may be that the execution of the function is
wrapped in a transaction anyway, so it's not really clear what WITH
HOLD would do for you.  Now, if we could decouple those things... THAT
would be exciting, for a lot more reasons than just this.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company