Re: Materialized View Patch File
От | Neel Patel |
---|---|
Тема | Re: Materialized View Patch File |
Дата | |
Msg-id | CAMcbDBHhk-F3i8OSU8Dg3J36uTogmj7UDs0vcf_n6Qpgn2Kw_w@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Materialized View Patch File (Dave Page <dave.page@enterprisedb.com>) |
Ответы |
Re: Materialized View Patch File
|
Список | pgadmin-hackers |
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,
Neel Patel
On Fri, May 31, 2013 at 5:59 PM, Dave Page <dave.page@enterprisedb.com> wrote:
On Wed, May 29, 2013 at 10:40 AM, Neel Patel
<neel.patel@enterprisedb.com> wrote:
> Hi Dave,
>> Please find the updated patch after fixing some of the issues and commentsI haven't looked at the code, but judging by the error I got when
> given by Ashesh.
first clicking on a view on a PG 9.2 server, you forgot to make this
code version-dependent:
2013-05-31 13:25:57 QUERY : Scalar query (localhost:5432): SELECT
count(*) FROM pg_matviews WHERE matviewname = 'pem.avail_servers' AND
schemaname = 'pem'
2013-05-31 13:25:57 ERROR : ERROR: relation "pg_matviews" does not exist
LINE 1: SELECT count(*) FROM pg_matviews WHERE matviewname = 'pem.av...
^
It then goes on to give the view definition as:
-- MATERIALIZED View: pem.avail_servers
-- DROP MATERIALIZED VIEW pem.avail_servers;
CREATE MATERIALIZED VIEW pem.avail_servers AS
SELECT s.id, s.description, s.server, s.port, s.database, s.ssl, s.serviceid,
...
...
(as a side note, the first line for a real mat view should be: "--
Materialized View: pem.avail_servers")
--
Dave Page
Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
Вложения
В списке pgadmin-hackers по дате отправления: