Hi,

> I wanted to hook into the EXPLAIN output for queries and add some extra 
> information, but since there is no standard_ExplainOneQuery() I had to copy 
> the code and create my own version.
>
> Since the pattern with other hooks for a function WhateverFunction() seems to 
> be that there is a standard_WhateverFunction() for each 
> WhateverFunction_hook, I created a patch to follow this pattern for your 
> consideration.
>
> I was also considering adding a callback so that you can annotate any node 
> with explanatory information that is not a custom scan node. This could be 
> used to propagate and summarize information from custom scan nodes, but I had 
> no immediate use for that so did not add it here. I would still be interested 
> in hearing if you think this is something that would be useful to the 
> community.

Thanks for the patch. LGTM.

I registered the patch on the nearest open CF [1] and marked it as
RfC. It is a pretty straightforward refactoring.

[1]: https://commitfest.postgresql.org/48/4879/

-- 
Best regards,
Aleksander Alekseev


Reply via email to