Re: pgsql: Modify pg_basebackup to use a new COPY subprotocol for base back

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: pgsql: Modify pg_basebackup to use a new COPY subprotocol for base back
Дата
Msg-id CA+TgmoZQfKPZXymiPXf4NyUEDzEjv5h6erW5ys1hySqDmHpN9w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgsql: Modify pg_basebackup to use a new COPY subprotocol for base back  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: pgsql: Modify pg_basebackup to use a new COPY subprotocol for base back  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Tue, Jan 18, 2022 at 8:55 PM Robert Haas <robertmhaas@gmail.com> wrote:
> Ah, I guess I copied and pasted the options wrong, or something.
> Anyway, I have an idea how to fix this. I didn't realize that we were
> going to read from the bbsink's buffer like this, and it's not
> properly aligned for that. I'll jigger things around to fix that.

Here's a patch. I'm still not able to reproduce the problem either
with the flags you propose (which don't cause a failure) or the ones
which Andres suggests (which make clang bitterly unhappy) or the ones
clang says I should use instead of the ones Andres suggests (which
make initdb fall over, so we never even get to the point of attempting
anything related to the code this patch modified).

Here's a patch, based in part on some off-list discussion with Andres.
I believe Andres has already confirmed that this fix works, but it
wouldn't hurt if Tom wants to verify it also.

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

Вложения

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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Add last commit LSN to pg_last_committed_xact()
Следующее
От: Tom Lane
Дата:
Сообщение: Re: pgsql: Modify pg_basebackup to use a new COPY subprotocol for base back