Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-20 Thread Guillaume Lelarge
2014-07-15 0:27 GMT+02:00 Dave Page : > 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 Lela

[pgadmin-hackers] Re: [pgadmin-hackers] Re: 答复: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-20 Thread Guillaume Lelarge
2014-07-15 9:21 GMT+02:00 Dave Page : > > > > On Tue, Jul 15, 2014 at 2:45 AM, liuyuanyuan > 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

[pgadmin-hackers] Re: 答复: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-15 Thread Dave Page
gadmin-hackers-ow...@postgresql.org [mailto: > pgadmin-hackers-ow...@postgresql.org] *代表 *Dave Page > *发送时间:* 2014年7月15日 6:28 > *收件人:* Guillaume Lelarge > *抄送:* pgadmin-hackers@postgresql.org > *主题:* Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a > sequence > > &

Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-14 Thread Dave Page
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 wrote: > > 2014-07-14 23:09 GMT+02:00 Dave

Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-14 Thread Guillaume Lelarge
2014-07-14 23:09 GMT+02:00 Dave Page : > 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 abou

Re: [pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-14 Thread Dave Page
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? -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK:http://www.enterprisedb.com The Enterprise PostgreSQL Company

[pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-14 Thread Guillaume Lelarge
Fix SQL for GRANT on a sequence Report from liuyuanyuan. Branch -- master Details --- http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=2b20c5fff3351a8e4a4066249e49f8731ea9d912 Modified Files -- CHANGELOG |2 ++ pgadmin/dlg/dlgSequence.cpp

[pgadmin-hackers] pgAdmin III commit: Fix SQL for GRANT on a sequence

2014-07-14 Thread Guillaume Lelarge
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/dlg