Fix misleading references to columns in GRANT/REVOKE summaries
Folks,
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-level GRANT/REVOKE, which
we don't have yet.
Cheers,
D
--
David Fetter <david@fetter.org> http://fetter.org/
phone: +1 415 235 3778 AIM: dfetter666
Skype: davidfetter
Remember to vote!
Consider donating to PostgreSQL: http://www.postgresql.org/about/donate
Attachments:
fix_grant_revoke.difftext/plain; charset=us-asciiDownload+6-6
David Fetter <david@fetter.org> 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-level GRANT/REVOKE, which
we don't have yet.
Apparently it's so misleading that you didn't understand it either.
The entire *point* of that paragraph is that we don't have the
feature. This proposed change is surely not an improvement...
regards, tom lane
Tom Lane wrote:
David Fetter <david@fetter.org> 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-level GRANT/REVOKE, which
we don't have yet.Apparently it's so misleading that you didn't understand it either.
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.
Show quoted text
regards, tom lane
---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match
Russell Smith <mr-russ@pws.com.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".
regards, tom lane
Tom Lane wrote:
Russell Smith <mr-russ@pws.com.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@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachments:
/rtmp/difftext/x-diffDownload+21-21
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 implementation and don't know what it means.
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
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 in our documentation.
That doesn't mean this one isn't useful. Please revert this.
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
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 don't see any other cases of
this in our documentation.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.
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Bruce Momjian <bruce@momjian.us> 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 describe or show the syntax in some informal
form, so that it didn't look like one of the <synopsis> sections we use
for supported syntax. I'm not sure what that would look like exactly,
but I do see merit in both sides of this discussion...
regards, tom lane
Tom Lane wrote:
Bruce Momjian <bruce@momjian.us> 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 describe or show the syntax in some informal
form, so that it didn't look like one of the <synopsis> sections we use
for supported syntax. I'm not sure what that would look like exactly,
but I do see merit in both sides of this discussion...
I am all for us describing how we don't match the SQL spec, but just
showing the syntax doesn't seem to help people understand how we don't
match the spec, does it? Are there more details to column-level GRANT
except saying we don't support it?
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +