[PATCH] Make ExplainBeginGroup()/ExplainEndGroup() public.

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: Raw Message | Whole Thread | 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

Responses

Browse pgsql-hackers by date

  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