Robert Haas writes:
> On Tue, Jul 25, 2017 at 9:54 PM, Kyotaro HORIGUCHI
> wrote:
>> The patch is a kind of straightforward and looks fine for me.
> +1 for this change.
LGTM too, pushed.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgres
On Tue, Jul 25, 2017 at 9:54 PM, Kyotaro HORIGUCHI
wrote:
> The patch is a kind of straightforward and looks fine for me.
+1 for this change.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postg
At Fri, 21 Jul 2017 10:09:25 -0400, Hadi Moshayedi wrote
in
> 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
Title should have been "Make ExplainOpenGroup()/ExplainCloseGroup()
public.".
Sorry for the misspell.
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