From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch] - RM 4030 - IDENTITY column not recognised |
Date: | 2019-03-20 15:07:46 |
Message-ID: | CA+OCxozdLgEHaytFemWpJ2pJwzx9GXZK-5ZxrOgGncbtkJhYKg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Wed, Mar 20, 2019 at 7:30 AM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi,
>
> Please find the attached patch to fix the RM #4030 - IDENTITY column not
> recognised.
> - Added support for IDENTITY column for PostgreSQL >= 10.0
>
A few issues unfortunately:
- There's an extra space in the generated SQL for an identity column on a
table, right before the end comma (in both the CREATE and reverse
engineered SQL.
- I cannot make an IDENTITY column a primary key through the UI, nor does
it reverse-engineer that property in the SQL if I create it via SQL (it
does properly set the switch value though).
- After creating an IDENTITY column, there should be a dependency on the
sequence, but I don't see this listed.
- We should consider the auto-created sequence a system object, and hide it
in the treeview by default as it's an implementation detail.
- If I click on an IDENTITY column in the treeview, the reverse-engineered
SQL just shows the plain datatype.
- Can we reasonably support the sequence_options clause?
Thanks.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Haoran Yu | 2019-03-20 15:26:28 | Re: Interested in GSoC projects on pgAdmin 4 |
Previous Message | Akshay Joshi | 2019-03-20 14:52:07 | pgAdmin 4 commit: Ensure that parser should be set in FTS Configuration |