Misleading comment in prologue of ReorderBufferQueueMessage

Поиск
Список
Период
Сортировка
От Ashutosh Bapat
Тема Misleading comment in prologue of ReorderBufferQueueMessage
Дата
Msg-id CAGEoWWTTzNzHOi8bj0wfAo1siGi-YEh6wqH1oaz4DrkTJ6HbTQ@mail.gmail.com
обсуждение исходный текст
Ответы Re: Misleading comment in prologue of ReorderBufferQueueMessage  (Amit Kapila <amit.kapila16@gmail.com>)
Список pgsql-hackers
The name of the function suggests that the given message will be queued in ReorderBuffer. The prologue of the function says so too
 776 /*
 777  * Queue message into a transaction so it can be processed upon commit.
 778  */
It led me to think that a non-transactional message is processed along with the surrounding transaction, esp. when it has an associated xid.

But in reality, the function queues only a transactional message and decoders a non-transactional message immediately without waiting for a commit.

We should modify the prologue to say
"Queue a transactional message into a transaction so that it can be processed upon commit. A non-transactional message is processed immediately." and also change the name of the function to ReorderBufferProcessMessage(), but the later may break API compatibility.

--
Best Wishes,
Ashutosh

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

Предыдущее
От: "Andrey V. Lepikhov"
Дата:
Сообщение: Re: [POC] Fast COPY FROM command for the table with foreign partitions
Следующее
От: Seino Yuki
Дата:
Сообщение: Re: Feature improvement for pg_stat_statements