Re: Missing HashAgg EXPLAIN ANALYZE details for parallel plans

On 2020-Jun-19, David Rowley wrote:

> > + size = offsetof(SharedAggInfo, sinstrument)
> > + + pcxt->nworkers * sizeof(AggregateInstrumentation);
> >
> > => There's a couple places where I'd prefer to see "+" at the end of the
> > preceding line (but YMMV).
>
> I pretty much just copied the whole of that code from nodeSort.c. I'm
> more inclined to just keep it as similar to that as possible. However,
> if pgindent decides otherwise, then I'll go with that. I imagine it
> won't move it though as that code has already been through indentation
> a few times before.

pgindent won't change your choice here. This seems an ideological
issue; each committer has got their style. Some people prefer to put
operators at the end of the line, others do it this way.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services