Horton

RE: Remove obsolate comments from 047_checkpoint_physical_slot

Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>

View thread View on postgr.es
From: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>
To: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>, "'pgsql-hackers@lists.postgresql.org'" <pgsql-hackers@lists.postgresql.org>
Cc: Alexander Korotkov <aekorotkov@gmail.com>
Date: 2025-09-25T07:23:33Z
Lists: pgsql-hackers 
Dear hackers,

> While reviewing others, I found $SUBJECT.

I found another cleanup point related with this. In CreateCheckPoint():

```
#ifdef USE_INJECTION_POINTS
	INJECTION_POINT("checkpoint-before-old-wal-removal");
#endif
```

Here USE_INJECTION_POINTS check is not needed. If the feature is disabled,
the macro function would be ((void) name). IIUC, we are using the macro if if-branch
exists.

E.g., another injection point "create-restart-point" does not have the #ifdef part.

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Commits

  1. Remove preprocessor guards from injection points

  2. Fix comments in recovery tests