From: Bruce Momjian <bruce@momjian.us>
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: protodef@gmail.com
Cc: pgsql-docs@lists.postgresql.org
Subject: Re: optionally schema-qualified for table_name
Date: Tue, 24 Mar 2020 16:58:18 -0400
Message-ID: <20200324205818.GA32258@momjian.us> (raw)
In-Reply-To: <af0208b5-7811-955b-3ddf-d5839c4a4798@2ndquadrant.com>
References: <158399268229.5708.13296960177949398026@wrigleys.postgresql.org>
<20200322001310.GA2031@momjian.us>
<22707.1584903901@sss.pgh.pa.us>
<20200322201518.GB2031@momjian.us>
<4628.1584915604@sss.pgh.pa.us>
<20200323012757.GC2031@momjian.us>
<af0208b5-7811-955b-3ddf-d5839c4a4798@2ndquadrant.com>
On Tue, Mar 24, 2020 at 09:35:25PM +0100, Peter Eisentraut wrote:
> On 2020-03-23 02:27, Bruce Momjian wrote:
> > On Sun, Mar 22, 2020 at 06:20:04PM -0400, Tom Lane wrote:
> > > Bruce Momjian <bruce@momjian.us> writes:
> > > > On Sun, Mar 22, 2020 at 03:05:01PM -0400, Tom Lane wrote:
> > > > > I don't really think this is an improvement, mainly because that
> > > > > error message is inventing a notation that we do not use in any
> > > > > other error message.
> > >
> > > > What do you suggest? The current message is:
> > >
> > > > Specify OWNED BY table.column or OWNED BY NONE.
> > >
> > > Yeah, and I think that's okay as-is, or at least we can't make it better
> > > without fairly whole-sale changes of our documentation practices.
> > > The fact that a table name can be schema-qualified is usually implicit,
> > > and I don't see why this place cries out for making it explicit
> > > more than other places. You could as well complain that there's
> > > nothing explicit here about double-quoting practices.
> >
> > OK, I will do just the documentation patch for this then.
>
> The same criticism applies to the documentation patch, I think. We don't
> usually make the schema part explicit.
That is a good point. I used CREATE VIEW as an example because that is
what the user reported, but it seems only create_view and reindexed use
a schema name qualification:
$ grep -l '<replaceable>schema</replaceable>' *.sgml
create_view.sgml
reindexdb.sgml
The reindexdb use is because of -S (reindex schema), which makes sense.
The create view case is used in an example of CREATE RECURSIVE VIEW and
should probably be removed.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EnterpriseDB https://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: bruce@momjian.us, peter.eisentraut@2ndquadrant.com, tgl@sss.pgh.pa.us, protodef@gmail.com, pgsql-docs@lists.postgresql.org
Subject: Re: optionally schema-qualified for table_name
In-Reply-To: <20200324205818.GA32258@momjian.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox