Horton

Re: Hooking into ExplainOneQuery() complicated by missing standard_ExplainOneQuery

Jelte Fennema-Nio <postgres@jeltef.nl>

View thread View on postgr.es
From: Jelte Fennema-Nio <postgres@jeltef.nl>
To: Mats Kindahl <mats@timescale.com>
Cc: Andrei Lepikhov <a.lepikhov@postgrespro.ru>, Michael Paquier <michael@paquier.xyz>, Aleksander Alekseev <aleksander@timescale.com>, pgsql-hackers@lists.postgresql.org
Date: 2024-03-06T09:32:01Z
Lists: pgsql-hackers 
This patch would definitely be useful for Citus. We indeed currently
copy all of that code into our own explain hook. And it seems we
actually have some bug. Because the es->memory branches were not
copied (probably because this code didn't exist when we copied it).

https://github.com/citusdata/citus/blob/d59c93bc504ad32621d66583de6b65f936b0ed13/src/backend/distributed/planner/multi_explain.c#L1248-L1289

Commits

  1. Improve support for ExplainOneQuery() hook