Re: Some regression tests for the pg_control_*() functions

2022-10-26 Thread Michael Paquier
On Wed, Oct 26, 2022 at 01:41:12PM +0530, Bharath Rupireddy wrote: > We will have bigger problems when a backend corrupts the pg_control > file, no? The bigger problems could be that the server won't come up > or it behaves abnormally or some other. Possibly, yes. > Can't the CRC check detect any

Re: Some regression tests for the pg_control_*() functions

2022-10-26 Thread Bharath Rupireddy
On Wed, Oct 26, 2022 at 12:48 PM Michael Paquier wrote: > > On Wed, Oct 26, 2022 at 10:13:29AM +0530, Bharath Rupireddy wrote: > > +1 for improving the test coverage. Is there a strong reason to > > validate individual output columns rather than select count(*) > 0 > > from pg_control_(); sort

Re: Some regression tests for the pg_control_*() functions

2022-10-26 Thread Michael Paquier
On Wed, Oct 26, 2022 at 10:13:29AM +0530, Bharath Rupireddy wrote: > +1 for improving the test coverage. Is there a strong reason to > validate individual output columns rather than select count(*) > 0 > from pg_control_(); sort of tests? If the intention is to validate > the pg_controlfile con

Re: Some regression tests for the pg_control_*() functions

2022-10-25 Thread Bharath Rupireddy
On Tue, Oct 25, 2022 at 11:07 AM Michael Paquier wrote: > > Hi all, > > As mentioned in [1], there is no regression tests for the SQL control > functions: pg_control_checkpoint, pg_control_recovery, > pg_control_system and pg_control_init. > > It would be minimal to check their execution, as of a

Some regression tests for the pg_control_*() functions

2022-10-24 Thread Michael Paquier
Hi all, As mentioned in [1], there is no regression tests for the SQL control functions: pg_control_checkpoint, pg_control_recovery, pg_control_system and pg_control_init. It would be minimal to check their execution, as of a "SELECT FROM func()", still some validation can be done on its output a