Re: auto_explain: Include JIT output if applicable

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: auto_explain: Include JIT output if applicable
Дата
Msg-id 20180929051110.sv65qp5pigawhvhg@alap3.anarazel.de
обсуждение исходный текст
Ответ на RE: auto_explain: Include JIT output if applicable  ("Shinoda, Noriyoshi (PN Japan GCS Delivery)" <noriyoshi.shinoda@hpe.com>)
Ответы RE: auto_explain: Include JIT output if applicable  ("Shinoda, Noriyoshi (PN Japan GCS Delivery)" <noriyoshi.shinoda@hpe.com>)
Список pgsql-hackers
Hi,

On 2018-09-29 05:04:25 +0000, Shinoda, Noriyoshi (PN Japan GCS Delivery) wrote:
> I tried this feature.
> I think that 'if (es->costs)' of the source code auto_explain.c will always be ‘true’.
> 
> Because it is not changed after 'es-> cost = true' in NewExplainState () function several rows ago.
> So I attached a patch to delete this if statement.

I think it's better to stay closer to what explain.c itself is doing -
it's not like that if statement costs us anything really...

- Andres


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

Предыдущее
От: "Shinoda, Noriyoshi (PN Japan GCS Delivery)"
Дата:
Сообщение: RE: auto_explain: Include JIT output if applicable
Следующее
От: "Shinoda, Noriyoshi (PN Japan GCS Delivery)"
Дата:
Сообщение: RE: auto_explain: Include JIT output if applicable