In comment.sgml:

COMMENT ON AGGREGATE my_aggregate (double precision) IS 'Computes
sample variance';

this raises error. However, 

COMMENT ON AGGREGATE my_aggregate double precision IS 'Computes
sample variance';

works but looks strange syntax. Should we fix the program or docs?
--
Tatsuo Ishii

Mon, 10 Sep 2001 15:40:16 +0900
Tatsuo Ishii <t-ishii@sra.co.jp>
COMMENT ON
Tatsuo Ishii writes:

> In comment.sgml:
>
> COMMENT ON AGGREGATE my_aggregate (double precision) IS 'Computes
> sample variance';
>
> this raises error. However,
>
> COMMENT ON AGGREGATE my_aggregate double precision IS 'Computes
> sample variance';
>
> works but looks strange syntax. Should we fix the program or docs?

I vote for fixing the program.

-- 
Peter Eisentraut   peter_e@gmx.net   http://funkturm.homeip.net/~peter


Tue, 11 Sep 2001 16:41:09 +0200 (CEST)
Peter Eisentraut <peter_e@gmx.net>
Re: COMMENT ON
Peter Eisentraut <peter_e@gmx.net> writes:
>> COMMENT ON AGGREGATE my_aggregate double precision IS 'Computes
>> sample variance';
>> 
>> works but looks strange syntax. Should we fix the program or docs?

> I vote for fixing the program.

If we fix this, we should also change DROP AGGREGATE, which also uses
the paren-less syntax.  (I think the COMMENT ON syntax was modeled on
DROP.)

I'd be in favor of changing, but we do need to maintain consistency.

Another issue is that pg_dump knows about using both of these
commands... we'll have a compatibility problem if we don't continue
to accept the old syntax for awhile.

			regards, tom lane

Tue, 11 Sep 2001 11:50:31 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: COMMENT ON 
> Peter Eisentraut <peter_e@gmx.net> writes:
> COMMENT ON AGGREGATE my_aggregate double precision IS 'Computes
> sample variance';
> 
> works but looks strange syntax. Should we fix the program or docs?

>> I vote for fixing the program.

> If we fix this, we should also change DROP AGGREGATE, which also uses
> the paren-less syntax.  (I think the COMMENT ON syntax was modeled on
> DROP.)

> I'd be in favor of changing, but we do need to maintain consistency.

> Another issue is that pg_dump knows about using both of these
> commands... we'll have a compatibility problem if we don't continue
> to accept the old syntax for awhile.

All fixed ...

			regards, tom lane

Wed, 03 Oct 2001 16:56:28 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Re: COMMENT ON 