Horton

Re: Remove redundant Assert(PgArchPID == 0); in PostmasterStateMachine

Fujii Masao <masao.fujii@oss.nttdata.com>

View thread View on postgr.es
From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Michael Paquier <michael@paquier.xyz>, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-07-15T02:49:33Z
Lists: pgsql-hackers 

On 2021/07/15 11:21, Michael Paquier wrote:
> On Wed, Jul 14, 2021 at 11:38:59PM +0530, Bharath Rupireddy wrote:
>> It looks like the commit d75288fb [1] added an unnecessary
>> Assert(PgArchPID == 0); in PostmasterStateMachine as the if block code
>> gets hit only when PgArchPID == 0. PSA small patch.

Good catch, Thanks!

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

Commits

  1. Remove unnecessary assertion in postmaster.c

  2. Make archiver process an auxiliary process.