Re: pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-15 0:27 GMT+02:00 Dave Page <dpage(at)pgadmin(dot)org>:

> Minor brain failure, but I believe the same comment applies.
>
>
Yeah, the code checks the version. If less than 8.3, it uses the old syntax
(GRANT ON TABLE). Otherwise, it uses the new syntax (GRANT ON SEQUENCE).

> On 14 Jul 2014, at 22:13, Guillaume Lelarge <guillaume(at)lelarge(dot)info>
> wrote:
>
> 2014-07-14 23:09 GMT+02:00 Dave Page <dpage(at)pgadmin(dot)org>:
>
>> That syntax was (once) intentional, and was not a bug, as earlier
>> versions of PG don't support ALTER SEQUENCE. Did you check the version
>> compatibility?
>>
>>
> I'm not sure I understand what you mean. We're not talking about ALTER
> SEQUENCE. We're talking about GRANT ON SEQUENCE.
>
>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK:http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On 14 Jul 2014, at 22:03, Guillaume Lelarge <guillaume(at)lelarge(dot)info>
>> wrote:
>>
>> Fix SQL for GRANT on a sequence
>>
>> Report from liuyuanyuan.
>>
>> Branch
>> ------
>> REL-1_18_0_PATCHES
>>
>> Details
>> -------
>>
>> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=5eb8e16348c001708d4b2d511eae988724a84938
>>
>> Modified Files
>> --------------
>> CHANGELOG | 2 ++
>> pgadmin/dlg/dlgSequence.cpp | 2 +-
>> pgadmin/schema/pgSequence.cpp | 2 +-
>> 3 files changed, 4 insertions(+), 2 deletions(-)
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>
>
> --
> Guillaume.
> http://blog.guillaume.lelarge.info
> http://www.dalibo.com
>
>

--
Guillaume.
http://blog.guillaume.lelarge.info
http://www.dalibo.com