Re: Allow COMMENT ON to accept an expression rather than just a string - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: Allow COMMENT ON to accept an expression rather than just a string
Date
Msg-id 162867790904111029k7a4c4d8diff0d4f5f1cd4f63b@mail.gmail.com
Whole thread Raw
In response to Re: Allow COMMENT ON to accept an expression rather than just a string  (Bruce Momjian <bruce@momjian.us>)
Responses Re: Allow COMMENT ON to accept an expression rather than just a string  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
2009/4/11 Bruce Momjian <bruce@momjian.us>:
> Tom Lane wrote:
>> Abhijit Menon-Sen <ams@oryx.com> writes:
>> > There's a TODO item about making COMMENT ON accept an expression. The
>> > grammar change is simple (SConst|NULL_P->a_expr), but as far as I can
>> > see, there are no similar utility commands that take expressions, and
>> > I'm not very familiar with the planner and executor, so I could use
>> > some advice about how to evaluate the expression.
>>
>> There aren't *any* utility commands that take expressions, and I would
>> say that that TODO item is seriously mis-scoped.  What is the use of
>> making COMMENT in particular do this?  Fixing all of them might be
>> interesting.
>
> I remember adding the TODO after a request from the user, but I have not
> seen further requests.  I have remove the item;  let's see if we get any
> further requests for it.

I thing so this TODO point is little bit step in bad direction. The
people use comment for storing some info about database object, that
should be available from dictionary. Typical use case is info about
last modification or create time. Now I am working in bigger
organisation and I see, so this informations are very important. So
well solution is add some table to dictionary, that store "create
timestamp, modification timestamp (ALTER), version (number of
changes)" to every database object where this has sense. Comment
should be stable and only text.

regards
Pavel Stehule

>
> --
>  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
>  EnterpriseDB                             http://enterprisedb.com
>
>  + If your life is a hard drive, Christ can be your backup. +
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>


pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: Patch for 8.5, transformationHook
Next
From: Tom Lane
Date:
Subject: Re: Allow COMMENT ON to accept an expression rather than just a string