| From: | Hadi Moshayedi <hadi(at)citusdata(dot)com> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | [PATCH] Make ExplainBeginGroup()/ExplainEndGroup() public. |
| Date: | 2017-07-21 14:09:25 |
| Message-ID: | CA+_kT_cCew4OKqTL-ueNndEL1jWYu+3e9RdLZP-_WR1KbONzfA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello,
The attached patch moves declarations of
ExplainBeginGroup()/ExplainEndGroup() 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]. But it
achieves it by having a copy of
ExplainBeginGroup()/ExplainEndGroup() 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
Thanks,
Hadi
| Attachment | Content-Type | Size |
|---|---|---|
| make_explainbeginendgroup_public.patch | text/x-patch | 2.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Joshua D. Drake | 2017-07-21 14:21:10 | Re: autovacuum can't keep up, bloat just continues to rise |
| Previous Message | Yugo Nagata | 2017-07-21 13:58:51 | Re: [PATCH] A hook for session start |