From: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Marko Tiikkaja <pgmail(at)joh(dot)to>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Materialized views WIP patch |
Date: | 2013-02-19 22:47:24 |
Message-ID: | 1361314044.27824.YahooMailNeo@web162904.mail.bf1.yahoo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Kevin Grittner <kgrittn(at)ymail(dot)com>
> There was one minor syntax issue not addressed by Noah, nor much
> discussed in general that I didn't want to just unilaterally
> choose; but given that nobody seems to care that much I will put
> forward a proposal and do it that way tomorrow if nobody objects.
> Before this patch tables were the only things subject to
> truncation, but now materialized views can also be truncated. So
> far we have been treating TABLE as a noise word in the truncate
> command. I assume we still want to allow tables to be truncated
> with or without the word. The question is what to do about
> materialized views, and wheter both can be specified on a single
> TRUNCATE statement. I propose that we allow TABLE or MATERIALIZED
> VIEW to be specified, or that part of the statement to be left out.
> I propose that either type of object be allowed unless one or the
> other is specified and the object to be truncated is not of that
> kind. So you could mix both kinds on one statement, so long as you
> didn't specify either kind.
When I went to do this, I hit a shift/reduce conflict, because with
TABLE being optional it couldn't tell whether:
TRUNCATE MATERIALIZED VIEW x, y, z;
... was looking for five relations or three. That goes away with
MATERIALIZED escalated to TYPE_FUNC_NAME_KEYWORD. Is that OK?
--
Kevin Grittner
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2013-02-19 23:32:51 | Re: Materialized views WIP patch |
Previous Message | Josh Berkus | 2013-02-19 22:45:10 | Re: Materialized views WIP patch |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2013-02-19 22:48:08 | Re: posix_fadvise missing in the walsender |
Previous Message | Josh Berkus | 2013-02-19 22:45:10 | Re: Materialized views WIP patch |