Re: A new function to wait for the backend exit after termination

От

Muhammad Usama

Тема

Дата

30 ноября 2020 г. в 14:39:31

Msg-id

160674717155.7563.13439910553399269588.pgcf@coridan.postgresql.org

обсуждение

Ответ на

Re: A new function to wait for the backend exit after termination (Bharath Rupireddy)

Список

pgsql-hackers

Дерево обсуждения

A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 21 октября 2020 г. в 13:02:15

Re: A new function to wait for the backend exit after termination Magnus Hagander <magnus@hagander.net> 21 октября 2020 г. в 13:13:36

Re: A new function to wait for the backend exit after termination Andres Freund <andres@anarazel.de> 22 октября 2020 г. в 02:35:25

Re: A new function to wait for the backend exit after termination "David G. Johnston" <david.g.johnston@gmail.com> 21 октября 2020 г. в 14:30:45

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 22 октября 2020 г. в 02:46:18

Re: A new function to wait for the backend exit after termination "David G. Johnston" <david.g.johnston@gmail.com> 22 октября 2020 г. в 03:09:35

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 22 октября 2020 г. в 04:12:33

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 28 октября 2020 г. в 11:50:46

Re: A new function to wait for the backend exit after termination Fujii Masao <masao.fujii@oss.nttdata.com> 28 октября 2020 г. в 13:11:42

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 31 октября 2020 г. в 10:58:18

Re: A new function to wait for the backend exit after termination Muhammad Usama <m.usama@gmail.com> 30 ноября 2020 г. в 14:39:31

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 2 декабря 2020 г. в 08:30:42

Re: A new function to wait for the backend exit after termination Muhammad Usama <m.usama@gmail.com> 2 декабря 2020 г. в 13:46:43

RE: A new function to wait for the backend exit after termination "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com> 3 декабря 2020 г. в 01:54:33

Re: A new function to wait for the backend exit after termination Tom Lane <tgl@sss.pgh.pa.us> 3 декабря 2020 г. в 02:01:34

RE: A new function to wait for the backend exit after termination "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com> 3 декабря 2020 г. в 02:21:51

RE: A new function to wait for the backend exit after termination "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com> 3 декабря 2020 г. в 03:32:46

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 3 декабря 2020 г. в 03:56:08

RE: A new function to wait for the backend exit after termination "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com> 4 декабря 2020 г. в 03:13:42

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 4 декабря 2020 г. в 06:29:51

RE: A new function to wait for the backend exit after termination "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com> 4 декабря 2020 г. в 08:31:40

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 4 декабря 2020 г. в 09:13:28

Re: A new function to wait for the backend exit after termination hou zhijie <houzj.fnst@cn.fujitsu.com> 4 декабря 2020 г. в 09:29:01

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 28 октября 2020 г. в 13:49:55

Re: A new function to wait for the backend exit after termination "David G. Johnston" <david.g.johnston@gmail.com> 28 октября 2020 г. в 14:21:09

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 29 октября 2020 г. в 05:13:57

Re: A new function to wait for the backend exit after termination "David G. Johnston" <david.g.johnston@gmail.com> 29 октября 2020 г. в 05:21:00

Re: A new function to wait for the backend exit after termination Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> 22 октября 2020 г. в 02:20:01

The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            not tested

I have tested the patch against current master branch (commit:6742e14959a3033d946ab3d67f5ce4c99367d332)
Both functions work without a problem and as expected.
Just a tiny comment/suggestion.
specifying a -ve timeout in pg_terminate_backed rightly throws an error, 
I am not sure if it would be right or a wrong approach but I guess we can ignore -ve
timeout in pg_terminate_backend function when wait (second argument) is false.

e.g.  pg_terminate_backend(12320, false,-1); -- ignore -1 timout since wait is false

The new status of this patch is: Ready for Committer

В списке pgsql-hackers по дате отправления

Предыдущее

От: Alvaro Herrera

Дата: 30 ноября 2020 г. в 14:37:38

Сообщение: Re: Printing LSN made easy

Следующее

От: Heikki Linnakangas

Дата: 30 ноября 2020 г. в 14:42:20

Сообщение: Re: Index Skip Scan (new UniqueKeys)