Re: Materialized View Patch File

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

In response to

Responses

Browse pgadmin-hackers by date

  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