On Tue, Nov 3, 2015 at 5:47 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> This comment doesn't make sense to me:
>
> + * (TSS_BUILDRUNS state prevents control reaching here in any
> + * case).
>
> Unless I'm missing something, that's not actually true.
It is true. consider_abort_common() only actually considers aborting
when state is TSS_INITIAL (we're still doing an internal sort). The
only other pertinent state here is TSS_BUILDRUNS. The point is that
TSS_BUILDRUNS is a generic "point of no return" past which
abbreviation cannot be aborted. That is a little arbitrary.
--
Peter Geoghegan