Re: Missing break in RelationFindReplTupleSeq

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: Missing break in RelationFindReplTupleSeq
Дата
Msg-id 20200131135831.GA1550@alvherre.pgsql
обсуждение исходный текст
Ответ на Missing break in RelationFindReplTupleSeq  (Konstantin Knizhnik <k.knizhnik@postgrespro.ru>)
Ответы Re: Missing break in RelationFindReplTupleSeq
Список pgsql-hackers
On 2020-Jan-31, Konstantin Knizhnik wrote:

> Eventually we find out that logical replication in the current version of
> Postgres works significantly slower on table with replica identity full than
> old pglogical implementation.
> 
> The comment to RelationFindReplTupleSeq says:
> 
>     Note that this stops on the first matching tuple.
> 
> But actually this function continue traversal until end of the table even if
> tuple was found.
> I wonder if break; should be added to the end of for loop.

Wow, you're right, and the "break" is missing there.  I propose it like
this.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Вложения

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

Предыдущее
От: Konstantin Knizhnik
Дата:
Сообщение: Missing break in RelationFindReplTupleSeq
Следующее
От: Hamid Akhtar
Дата:
Сообщение: Re: Do we need to handle orphaned prepared transactions in the server?