From: | Dave Page <dave(dot)page(at)enterprisedb(dot)com> |
---|---|
To: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Materialized View Patch File |
Date: | 2013-06-03 11:50:59 |
Message-ID: | CA+OCxoxZsp5P0Rm_Kihds49GYXCvBJS+bmDZgzMNexO97EYyEw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Mon, Jun 3, 2013 at 7:56 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> Thanks for update.
>
> Yes it is a bug, I have not put the version check.
>
> Below are the changes.
> --- Put the PG version check for mat views
> --- In SQL Pane window change the the description of the view from --
> MATERIALIZED View: xxx TO
> -- Materialized View: xxx
>
> Please find the updated patch in attachement.
Thanks - a few more comments now I've tested further:
- The tab should be called "Materialization" (unless someone has a better idea)
- I can still enable the "Materialized View" option on a 9.2 database
on the dialogue.
- I can't select a tablespace from the drop down menu, but I can type
a name in. Should be select-only - see dlgTable.
- The vacuum options UI design should match the existing one on
dlgTable (it's a nice design, but consistency comes first).
- Sidenote, because it will be irrelevant, s/Toastable/Toast table/
--
Dave Page
Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
From | Date | Subject | |
---|---|---|---|
Next Message | Neel Patel | 2013-06-03 18:29:02 | Re: Materialized View Patch File |
Previous Message | Neel Patel | 2013-06-03 06:56:37 | Re: Materialized View Patch File |