Re: [bug fix] Memory leak in dblink

Поиск
Список
Период
Сортировка
От MauMau
Тема Re: [bug fix] Memory leak in dblink
Дата
Msg-id 754FA21D1384477EAA86FE5F05004906@maumau
обсуждение исходный текст
Ответ на Re: [bug fix] Memory leak in dblink  (Joe Conway <mail@joeconway.com>)
Ответы Re: [bug fix] Memory leak in dblink
Список pgsql-hackers
From: "Joe Conway" <mail@joeconway.com>
> Fair enough -- this patch does it at that level in
> materializeQueryResult()

I'm in favor of this.  TBH, I did this at first, but I was afraid this would 
be rejected due to the reason mentioned earlier.

if statement in PG_TRY block is not necessary like this, because sinfo is 
zero-cleared.

  PG_TRY();  {
+   /* Create short-lived memory context for data conversions */
+   sinfo.tmpcontext = AllocSetContextCreate(CurrentMemoryContext,
+            "dblink temporary context",

Regards
MauMau




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

Предыдущее
От: Josh Berkus
Дата:
Сообщение: Re: idle_in_transaction_timeout
Следующее
От: Andres Freund
Дата:
Сообщение: Re: -DDISABLE_ENABLE_ASSERT