[PATCH] Make ExplainOpenGroup()/ExplainCloseGroup() available for extensions.

Started by Hadi Moshayedialmost 9 years ago1 messageshackers
Jump to latest
#1Hadi Moshayedi
hadi@citusdata.com

Hello Hackers,

The attached patch moves declarations of
ExplainOpenGroup()/ExplainCloseGroup() from explain.c to explain.h.

This can be useful for extensions that need explain groups in their
custom-scan explain output.

For example, Citus uses groups in its custom explain outputs [1]https://github.com/citusdata/citus/blob/master/src/backend/ distributed/planner/multi_explain.c. But it
achieves it by having a copy of ExplainOpenGroup()/ExplainCloseGroup() in
its source code, which is not the best way.

Please review.

[1]: https://github.com/citusdata/citus/blob/master/src/backend/ distributed/planner/multi_explain.c
distributed/planner/multi_explain.c

Thanks,
Hadi

Attachments:

make_explainopenclosegroup_public.patchtext/x-patch; charset=US-ASCII; name=make_explainopenclosegroup_public.patchDownload+7-6