From: | 权宗亮 <quanzongliang(at)gmail(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: little patch for "Detect serial columns from pg_depend" and bugfix |
Date: | 2012-09-05 14:48:40 |
Message-ID: | CAE7Kk4hTtcV1o2JiQYyLoVfyKMExN10BRe8ves5WkOt7MX1x9w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
2012/9/5 Dave Page <dpage(at)pgadmin(dot)org>
> On Wed, Sep 5, 2012 at 3:39 AM, Quan Zongliang <quanzongliang(at)gmail(dot)com>
> wrote:
> >
> > new patch attched.
> > I tested it under Cygwin, seems OK.
>
> I got an error on my first test I'm afraid :-(
>
> 2012-09-05 14:39:41 QUERY : Set query (localhost:5432): SELECT classid
> FROM pg_depend
> WHERE refobjid=207478::oid AND refobjsubid = 1 AND objid IN
> ('pem.agent_id_seq'::regclass,'"pem.agent_id_seq"'::regclass) AND
> deptype='a'
> 2012-09-05 14:39:41 ERROR : ERROR: relation "pem.agent_id_seq" does not
> exist
> LINE 3: ...id = 1 AND objid IN ('pem.agent_id_seq'::regclass,'"pem.agen...
>
> How does the attached version of the patch look? I've tweaked the
> formatting and removed the excess quotes that seemed to be causing the
> issue.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
There is a bug in original source.
I will try to fix it tomorrow.
When a column is given a special name.
The serial type can not be reversed correctly.
defination: "c 2" bigserial
display in pgAdmin: "c 2" bigint NOT NULL DEFAULT nextval('pem."t1_c
2_seq"'::regclass)
Because pgAdmin check columns which default value has 'pem.t1_c
2_seq'::regclass
and '"pem.t1_c 2_seq"'::regclass. But second one should be 'pem."t1_c
2_seq"'::regclass
Quan Zongliang
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2012-09-05 21:07:30 | pgAdmin III commit: Update chinese translation |
Previous Message | Dave Page | 2012-09-05 14:10:41 | Re: patch for "Sequence: calculate next value to be expected" |