On 2017-07-17 23:04:43 +0200, Julien Rouhaud wrote:
> On 17/07/2017 16:57, Andres Freund wrote:
> > The latter obviously isn't ready, but might make clearer what I'm
> > thinking about.
>
> It did for me, and FWIW I like this approach.
Cool.
> > If we were to go this route we'd have to probably move
> > the callback assignment into the ExecInit* routines, and possibly
> > replace the switch in ExecInitNode() with another callback, assigned in
> > make_*, and implement callbacks for ExecShutdown, MultiExecProcNode etc.
> >
> > This results in a good speedup in tpc-h btw:
> > master total min: 46434.897 cb min: 44778.228 [diff -3.70]
>
> Is it v11 material or is there any chance to make it in v10?
I think it'd make sense to apply the first to v10 (and earlier), and the
second to v11. I think this isn't a terribly risky patch, but it's
still a relatively large change for this point in the development
cycle. I'm willing to reconsider, but that's my default.
- Andres