On Wed, Dec 29, 2010 at 4:09 AM, Heikki Linnakangas
> On 29.12.2010 06:54, Robert Haas wrote:
>> With the patch:
>> rhaas=# cluster v;
>> ERROR: views do not support CLUSTER
> "do not support" sounds like a missing feature, rather than a nonsensical
> command. How about something like "CLUSTER cannot be used on views"
I'm fine with flipping the ordering around. I think I like it
marginally better this way, but you and Tom both seem to prefer the
opposite ordering, ergo so be it (barring a sudden influx of contrary
> The patch changes a bunch of heap_openrv() calls to relation_openrv().
> Perhaps it would be better make the error message something like "\"%s\" is
> not a table", and keep the callers unchanged. It's not particularly useful
> to repeat the command in the error message, the user should know what
> command he issued. Even if it's buried deep in a PL/pgSQL function or
> something, it should be clear from the context lines.
Did you read the whole thread?
The Enterprise PostgreSQL Company
In response to
pgsql-hackers by date
|Next:||From: Robert Haas||Date: 2010-12-29 11:35:23|
|Subject: Re: pg_streamrecv for 9.1?|
|Previous:||From: Magnus Hagander||Date: 2010-12-29 10:47:53|
|Subject: pg_streamrecv for 9.1?|