Re: [PATCHES] COMMENT ON patch - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: [PATCHES] COMMENT ON patch
Date
Msg-id Pine.LNX.4.10.9910242042040.377-100000@peter-e.yi.org
Whole thread Raw
Responses Re: [HACKERS] Re: [PATCHES] COMMENT ON patch  (Bruce Momjian <maillist@candle.pha.pa.us>)
List pgsql-hackers
On Oct 23, Mike Mascari mentioned:

> The following patch extends the COMMENT ON functionality to the
> rest of the database objects beyond just tables, columns, and views. The
> grammer of the COMMENT ON statement now looks like:
> 
> COMMENT ON [ 
>   [ DATABASE | INDEX | RULE | SEQUENCE | TABLE | TYPE | VIEW ] <objname> |
> 
>   COLUMN <relation>.<attribute> |
>   AGGREGATE <aggname> <aggtype> | 
>   FUNCTION <funcname> (arg1, arg2, ...) | 
>   OPERATOR <op> (leftoperand_typ rightoperand_typ) | 
>   TRIGGER <triggername> ON relname> 
> ] IS 'text'

In related news I'd like to point out that psql's \dd command now supports
aggregates, functions, operators, types, relations (tables, views,
indices, sequences), rules, and triggers. In addition all the other \d?
commands (\da, \df, \dT, \do, \dtvsiS), as well as \l, have comments
display switchable. Attribute comments can be seen in \d in a similar
fashion. You can also give a comment on \lo_import which can then be seen
in \lo_list (=\dl). Seems like all the bases are covered.

Just to confirm a few things here: Are you keying rule comments on
pg_rewrite.oid? Are operator comments keyed on the oid of the underlying
function? (Perhaps that could even be changed so you can put a comment on
the operator and a note like "implementation of %^*& operator" on the
function. Just a thought.)

Now we just have to stick a whole bunch of comments on all system stuff.
Where would be a good place to do this? Where are all the comments on the
built-in operators generated?
-Peter

-- 
Peter Eisentraut                  Sernanders vaeg 10:115
peter_e@gmx.net                   75262 Uppsala
http://yi.org/peter-e/            Sweden



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: mv backend/port ../../
Next
From: Tom Lane
Date:
Subject: Catalog version numbering added (committers READ THIS)