Thread: False comment about speculative insertion

False comment about speculative insertion

From
Peter Geoghegan
Date:
Attached patch removes a reference to an executor README section about
speculative insertion. In fact, the high-level overview of speculative
insertion ended up at the top of execIndexing.c. The executor README
was not touched by the ON CONFLICT patch at all.

I don't think it's necessary to refer to execIndexing.c within
ExecInsert instead. All the routines being called from ExecInsert()
that relate to speculative insertion are in execIndexing.c anyway.

--
Peter Geoghegan

Attachment

Re: False comment about speculative insertion

From
Heikki Linnakangas
Date:
On 07/26/2015 10:30 PM, Peter Geoghegan wrote:
> Attached patch removes a reference to an executor README section about
> speculative insertion. In fact, the high-level overview of speculative
> insertion ended up at the top of execIndexing.c. The executor README
> was not touched by the ON CONFLICT patch at all.
>
> I don't think it's necessary to refer to execIndexing.c within
> ExecInsert instead. All the routines being called from ExecInsert()
> that relate to speculative insertion are in execIndexing.c anyway.

Fixed, thanks.

- Heikki