Re: Extension Templates S03E11

From: Boszormenyi Zoltan <zb(at)cybertec(dot)at>
To: Dimitri Fontaine <dimitri(at)2ndQuadrant(dot)fr>
Cc: Thom Brown <thom(at)linux(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Extension Templates S03E11
Date: 2013-08-27 18:34:21
Message-ID: 521CF12D.5080408@cybertec.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2013-08-27 18:09 keltezéssel, Dimitri Fontaine írta:
> Hi,
>
> Boszormenyi Zoltan <zb(at)cybertec(dot)at> writes:
>> Here's a review for this patch.
> Thanks for that review Zoltan!

You're welcome.

>> I would like to see the control parameters documented in allcaps
>> in CREATE EXTENSION TEMPLATE. Then ALTER EXTENSION
>> TEMPLATE should reference the CREATE instead of the longer
>> text in <xref linkend="extend-extensions">. This xref can still
>> be there for reference in all three of CREATE/ALTER/DROP
>> EXTENSION TEMPLATE.
> I didn't follow exactly your proposal here because I didn't want to have
> to maintain the control parameter description list in two different
> places. I've still added a detailed list with references and details and
> more importantly with coverage of e.g. "NORELOCATABLE" which is not
> covered in the referenced material.

It looks better. Now that the lowercase keywords are expected
by the eye in the referenced text, they will be much easier to find.
Thanks.

Best regards,
Zoltán Böszörményi

--
----------------------------------
Zoltán Böszörményi
Cybertec Schönig & Schönig GmbH
Gröhrmühlgasse 26
A-2700 Wiener Neustadt, Austria
Web: http://www.postgresql-support.de
http://www.postgresql.at/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2013-08-27 18:47:49 Re: error out when building pg_xlogdump with pgxs
Previous Message David E. Wheeler 2013-08-27 18:32:36 Re: PL/pgSQL PERFORM with CTE