On Tue, Jul 15, 2014 at 2:45 AM, liuyuanyuan <[email protected]> wrote:
> Thanks for your attention to this topic! > > In fact, none pgadmin can be compatible with all versions of the database. > We've done pretty well so far, supporting right back to 7.3, through we don't officially support the much older versions now, but that's mostly due to the amount of testing it requires. > Although earlier versions of PG don't support GRANT ON SEQUENCE, > > but new versions support. So I think this adjustment is necessary . > Old versions supported GRANT ON TABLE for sequences iirc, (as sequences are members of pg_class). This change is only necessary if newer versions of PostgreSQL dropped that syntax. It may be desirable, if we no longer support any versions of PostgreSQL that needed the old syntax. > > > Jasmine > > > > *发件人:* [email protected] [mailto: > [email protected]] *代表 *Dave Page > *发送时间:* 2014年7月15日 6:28 > *收件人:* Guillaume Lelarge > *抄送:* [email protected] > *主题:* Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a > sequence > > > > Minor brain failure, but I believe the same comment applies. > > -- > 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:13, Guillaume Lelarge <[email protected]> > wrote: > > 2014-07-14 23:09 GMT+02:00 Dave Page <[email protected]>: > > 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 <[email protected]> > 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 ([email protected]) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > > > > > -- > > Guillaume. > > http://blog.guillaume.lelarge.info > > http://www.dalibo.com > > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
