Re: Decoding speculative insert with toast leaks memory

Поиск
Список
Период
Сортировка
От Dilip Kumar
Тема Re: Decoding speculative insert with toast leaks memory
Дата
Msg-id CAFiTN-t=pKW6eBPd7a566qc6MumE8PCY+1pGFOeVFSAhZZvxbw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Decoding speculative insert with toast leaks memory  (Amit Langote <amitlangote09@gmail.com>)
Ответы Re: Decoding speculative insert with toast leaks memory  (Amit Langote <amitlangote09@gmail.com>)
Список pgsql-hackers
On Thu, Jun 17, 2021 at 12:52 PM Amit Langote <amitlangote09@gmail.com> wrote:

> Oh I missed that the problem report is for the PG13 branch.
>
> How about the attached patch then?
>
Looks good, one minor comment, how about making the below comment,
same as on the head?

- if (!found || !entry->replicate_valid)
+ if (!found)
+ {
+ /*
+ * Make the new entry valid enough for the callbacks to look at, in
+ * case any of them get invoked during the more complicated
+ * initialization steps below.
+ */

On head:
if (!found)
{
/* immediately make a new entry valid enough to satisfy callbacks */

-- 
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com



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

Предыдущее
От: Paul Guo
Дата:
Сообщение: Re: Two patches to speed up pg_rewind.
Следующее
От: Heikki Linnakangas
Дата:
Сообщение: Re: Different compression methods for FPI