On Fri, Jun 11, 2021 at 09:37:50PM -0700, Noah Misch wrote:
> On Fri, Jun 11, 2021 at 08:54:08PM -0500, Justin Pryzby wrote:
> > On Sat, Jun 05, 2021 at 12:08:01PM -0700, Noah Misch wrote:
> > > > > My preference is to remove pg_wait_for_backend_termination(). The
> > > > > use case
> > > > > that prompted this thread used pg_terminate_backend(pid, 180000); it
> > > > > doesn't
> > > > > need pg_wait_for_backend_termination().
> >
> > Is this an Opened Issue ?
>
> An Open Item? Not really, since there's no objective defect. Nonetheless,
> the attached is what I'd like to use.
I think of this as a list of stuff to avoid forgetting that needs to be
addressed or settled before the release.
If the value of the new function is marginal, it may be good to remove it, else
we're committed to supporting it.
Even if it's not removed, the descriptions should be cleaned up.
| src/include/catalog/pg_proc.dat- descr => 'terminate a backend process and
if timeout is specified, wait for its exit or until timeout occurs',
=> I think doesn't need to change or mention the optional timeout at all
| src/include/catalog/pg_proc.dat-{ oid => '2137', descr => 'wait for a backend
process exit or timeout occurs',
=> should just say "wait for a backend process to exit". The timeout has a
default.
