| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Russell Smith <mr-russ(at)pws(dot)com(dot)au>, David Fetter <david(at)fetter(dot)org>, PostgreSQL Docs <pgsql-docs(at)postgresql(dot)org>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: [PATCHES] Fix misleading references to columns in GRANT/REVOKE summaries |
| Date: | 2007-04-07 03:49:04 |
| Message-ID: | 200704070349.l373n4l13436@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-docs pgsql-patches |
Tom Lane wrote:
> Russell Smith <mr-russ(at)pws(dot)com(dot)au> writes:
> > Tom Lane wrote:
> >> The entire *point* of that paragraph is that we don't have the
> >> feature. This proposed change is surely not an improvement...
> >>
> > Maybe removing the entire example would be more helpful. I don't find
> > it clear to have a command outline in a compatibility block.
>
> True, there doesn't seem to be any point in providing a full syntax
> summary rather than just saying "the SQL spec says you can grant
> privileges on columns but we don't support that yet".
Agreed. Patch attached and applied. I don't see any other cases of
this in our documentation.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
| Attachment | Content-Type | Size |
|---|---|---|
| /rtmp/diff | text/x-diff | 2.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2007-04-08 19:34:53 | Re: [PATCHES] Fix misleading references to columns in GRANT/REVOKE summaries |
| Previous Message | Tom Lane | 2007-04-07 02:24:57 | Re: [PATCHES] Fix misleading references to columns in GRANT/REVOKE summaries |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2007-04-07 04:19:28 | Re: UTF8MatchText |
| Previous Message | Bruce Momjian | 2007-04-07 03:39:40 | Re: HOT Patch - Ready for review |