Remove unnecessary assertion in postmaster.c

michaelpq · michaelpq · commit dc2db1eac365 · 2021-07-15T15:00:45.000+09:00
A code path asserted that the archiver was dead, but a check made that impossible to happen. Author: Bharath Rupireddy Discussion: https://postgr.es/m/CALj2ACW=CYE1ars+2XyPTEPq0wQvru4c0dPZ=Nrn3EqNBkksvQ@mail.gmail.com Backpatch-throgh: 14
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
@@ -3934,7 +3934,6 @@ PostmasterStateMachine(void)
 			Assert(CheckpointerPID == 0);
 			Assert(WalWriterPID == 0);
 			Assert(AutoVacPID == 0);
-			Assert(PgArchPID == 0);
 			/* syslogger is not considered here */
 			pmState = PM_NO_CHILDREN;
 		}

Original file line number	Diff line number	Diff line change
`@@ -3934,7 +3934,6 @@ PostmasterStateMachine(void)`
`3934`	`3934`	`Assert(CheckpointerPID == 0);`
`3935`	`3935`	`Assert(WalWriterPID == 0);`
`3936`	`3936`	`Assert(AutoVacPID == 0);`
`3937`		`- Assert(PgArchPID == 0);`
`3938`	`3937`	`/* syslogger is not considered here */`
`3939`	`3938`	`pmState = PM_NO_CHILDREN;`
`3940`	`3939`	`}`