Re: SQL:2011 Application Time Update & Delete

Peter Eisentraut <peter(at)eisentraut(dot)org> writes:
> I have committed the patches 0001 through 0003.

Coverity is complaining that rsi.isDone may be used uninitialized in
ExecForPortionOfLeftovers. It's correct: that function is not obeying
the function call protocol, and it's only accidental that it's not
failing. In ValuePerCall mode the caller is supposed to initialize
isDone (and isnull too) before each call. The canonical reference
for this is execSRF.c, and it does that. So I think we need something
like the attached.

I notice that execSRF.c also runs pgstat_init_function_usage and
pgstat_end_function_usage around each call. That's not too important
right now, but I wonder whether we should add it while we're looking
at this. It would perhaps be important once we support user-defined
withoutPortionProcs.

regards, tom lane