Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-08 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian <[EMAIL PROTECTED]> writes: > > Peter Eisentraut wrote: > >> That doesn't mean this one isn't useful. Please revert this. > > > Well, Tom and I thought it caused confusion, as did the person reporting > > the confusion. You saying to revert it isn't enough. > > A

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-08 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > Peter Eisentraut wrote: >> That doesn't mean this one isn't useful. Please revert this. > Well, Tom and I thought it caused confusion, as did the person reporting > the confusion. You saying to revert it isn't enough. A possible compromise is to descr

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-08 Thread Bruce Momjian
Peter Eisentraut wrote: > Bruce Momjian wrote: > > > 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

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-08 Thread Peter Eisentraut
Bruce Momjian wrote: > > 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

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-08 Thread Peter Eisentraut
Tom Lane wrote: > 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". I think it's pretty useful if people see a command of this form from some other implemen

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Bruce Momjian
Tom Lane wrote: > Russell Smith <[EMAIL PROTECTED]> 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 >

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Tom Lane
Russell Smith <[EMAIL PROTECTED]> 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

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Russell Smith
Tom Lane wrote: David Fetter <[EMAIL PROTECTED]> writes: Per a question Alexey Parshin asked in the IRC channel, I'm attaching a patch to the GRANT and REVOKE syntax summaries which replaces the misleading word "column" with "parameter." "Column" is misleading because it could be read to imp

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Tom Lane
David Fetter <[EMAIL PROTECTED]> writes: > Per a question Alexey Parshin asked in the IRC channel, I'm attaching > a patch to the GRANT and REVOKE syntax summaries which replaces the > misleading word "column" with "parameter." "Column" is misleading > because it could be read to imply a column-le