Re: ALTER EXTENSION UPGRADE, v3

Поиск
Список
Период
Сортировка
От Dimitri Fontaine
Тема Re: ALTER EXTENSION UPGRADE, v3
Дата
Msg-id 87sjvukirj.fsf@2ndQuadrant.fr
обсуждение исходный текст
Ответ на Re: ALTER EXTENSION UPGRADE, v3  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Tom Lane <tgl@sss.pgh.pa.us> writes:
> First off, I don't much care for the name "CREATE WRAPPER EXTENSION".
> WRAPPER is a misnomer in this case --- it's not wrapping anything.
> I think Dimitri stated that he chose WRAPPER just because it was an
> already existing keyword, but that isn't much of an excuse.

Let's get rid of the two-stages idea now that we have proper VERSION
support in the commands, as seen in another email.

> Second, I don't like anything about the term "null version" for the
> case of bootstrapping from an old-style contrib module.  Null implies
> unknown, which isn't what we've got here --- the upgrade script is going

Yes it's what we have, the way I see it at least.  The version number
certainly exists, it's just that PostgreSQL had no way to know about it
until now.  Certainly that concept can be called unknown…

> One minor objection to this idea is that "foo--1.0.sql" looks more like a
> typo than anything else.  We could alternatively decide that the special
> reserved version name is '0', so that bootstrap script names look like
> "foo-0-1.0.sql".  But if you don't want to have any built-in assumptions
> about what version names mean, you might not like that idea.

I hope you will like the version aliases proposal I've been making in
other emails, just saying it again as a loose cross-reference :)

> Third, I'm also not thrilled with the syntax "ALTER EXTENSION foo
> UPGRADE".  UPGRADE isn't an existing keyword (note that VERSION is).

Fair enough.

> And I don't see any strong reason to assume that the version change
> is an "upgrade".  Authors might well choose to support sidegrades or
> downgrades, especially with experimental modules.  I suggest either
>
>     ALTER EXTENSION foo UPDATE [ TO 'version' ]
>
>     ALTER EXTENSION foo VERSION [ 'version' ]
>
> the main excuse for the latter being that it's closer to the comparable
> syntax in CREATE EXTENSION.

I somehow would prefer a mix of those two proposals:
 ALTER EXTENSION foo TO VERSION 'version'; ALTER EXTENSION foo TO VERSION alias;

Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr     PostgreSQL : Expertise, Formation et Support


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Dimitri Fontaine
Дата:
Сообщение: Re: ALTER EXTENSION UPGRADE, v3
Следующее
От: Jan Urbański
Дата:
Сообщение: Re: pl/python custom exceptions for SPI