Re: pgsql: Move scanint8() to numutils.c

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: pgsql: Move scanint8() to numutils.c
Дата
Msg-id CA+TgmobLY=ZcvD9cLnhA92ATnEfNHgfEDM4MUCsRADH9td+mrw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgsql: Move scanint8() to numutils.c  (Joe Conway <mail@joeconway.com>)
Ответы Re: pgsql: Move scanint8() to numutils.c
Re: pgsql: Move scanint8() to numutils.c
Список pgsql-hackers
On Tue, Feb 15, 2022 at 10:39 AM Joe Conway <mail@joeconway.com> wrote:
> (moving to hackers)
>
> I guess shame on me for not noticing the thread, but I don't see any
> discussion about the potential for breakage to external projects.
>
> scanint8() is exported, and this change breaks at least two extensions I
> maintain.
>
> A quick scan (no pun intended ;-)) of github shows other potential
> breakage, including at least older (still open source) versions of
> pglogical.

I don't have a particularly strong view on whether the underlying
change was a good idea here, but the breakage you're talking about
seems pretty easy to fix, unless I'm missing something? I think it
would be a bad idea to make such a change in a minor release without
concern for extension breakage, but in a new major release it doesn't
seem like a problem to me.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



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

Предыдущее
От: Nathan Bossart
Дата:
Сообщение: Re: Avoid erroring out when unable to remove or parse logical rewrite files to save checkpoint work
Следующее
От: Matthias van de Meent
Дата:
Сообщение: Re: Lowering the ever-growing heap->pd_lower