From: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dave(dot)page(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Materialized View Patch File |
Date: | 2013-07-19 06:21:29 |
Message-ID: | CAMcbDBEn1+=fuADHaB7qLnz-+ZYL45kdcmM-tZM8hY3WYJCXBg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
Thanks for the comment.
We made the changes as you suggested.
Please find the updated patch in attachment and let me know if you have any
query.
Thanks,
Neel Patel
On Thu, Jul 18, 2013 at 5:54 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com>wrote:
> Hi
>
> I realised this code is all wonky anyway - why is IsMaterializedView()
> not just an accessor over a pgView private variable that is set in
> pgViewFactory::CreateObjects when we create the object? We already
> know at that point whether it's a matview or not (from
> pg_class.relkind). Querying pg_class every time we want to check if
> the view is materialized or not is horribly inefficient. There should
> be just one function, that is a member of pgView and takes no
> arguments. Whereever it's called, we just need to get the pgView
> object and call it.
>
> Please fix.
>
> Thanks.
>
> On Thu, Jul 18, 2013 at 5:58 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
> > Hi Dave,
> >
> > Please find the updated patch with modification in query.
> >
> > Thanks,
> > Neel Patel
> >
> >
> > On Wed, Jul 17, 2013 at 5:15 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com>
> > wrote:
> >>
> >> On Wed, Jul 17, 2013 at 12:23 PM, Neel Patel
> >> <neel(dot)patel(at)enterprisedb(dot)com> wrote:
> >> > Hi Dave,
> >> >
> >> > Just want to know.
> >> >
> >> > Now we are passing GetName() to qtDbString() and to form the complete
> >> > query
> >> > we need to use qtDbString() to compare the string.
> >> >
> >> > We have added below string.
> >> >
> >> > wxString sql = wxT("SELECT count(*) FROM pg_matviews WHERE
> matviewname =
> >> > ")
> >> > + qtDbString(this->GetName()) + wxT(" AND schemaname = ") +
> >> > qtDbString(this->GetSchema()->GetName());
> >> >
> >> > Is it correct ?
> >>
> >> I'd expect you to be using GetQuotedIdentifier() there. For a view and
> >> schema, the identifier is the name anyway.
> >>
> >>
> >> --
> >> Dave Page
> >> Chief Architect, Tools & Installers
> >> EnterpriseDB: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >>
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >
> >
>
>
>
> --
> Dave Page
> Chief Architect, Tools & Installers
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
Attachment | Content-Type | Size |
---|---|---|
MaterializedView_Bug_Fix_V3.patch | application/octet-stream | 6.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Richard PALO | 2013-07-19 06:51:24 | Fwd: patch proposal for acinclude.m4 to better support gcc on illumos/smartos |
Previous Message | Dave Page | 2013-07-18 20:28:11 | Re: Bug in the graphical explain |