On 2017-07-26 15:03:37 -0400, Tom Lane wrote: > Andres Freund <and...@anarazel.de> writes: > > I've moved the CHECK_FOR_INTERRUPTS() to the callsites. That > > unsurprisingly ends up being somewhat verbose, and there's a bunch of > > minor judgement calls where exactly to place them. While doing so I've > > also added a few extra ones. Did this in a separate patch to make it > > easier to review. > > Hm, that seems kinda backwards to me; I was envisioning the checks > moving to the callees not the callers. I think it'd end up being > about the same number of occurrences of CHECK_FOR_INTERRUPTS(), > and there would be less of a judgment call about where to put them.
Hm, that seems a bit riskier - easy to forget one of the places where we might need a CFI(). We certainly are missing a bunch of them in various nodes - I tried to add ones I saw as missing, but it's quite some code. Keeping them close to ExecProcNode() makes that call easier. I'm not quite seing how solely putting them in callees removes the judgement call issue? Greetings, Andres Freund -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
