Hi,

On Fri, Feb 16, 2024 at 12:32:45AM +0000, Zhijie Hou (Fujitsu) wrote:
> Agreed. Here is new patch set as suggested. I used debug2 in the 040 as it
> could provide more information about communication between primary and 
> standby.
> This also doesn't increase noticeable testing time on my machine for debug
> build.

Same here, and there is no big diff as far the amount of log generated:

Without the patch:

$ du -sh ./src/test/recovery/tmp_check/log/*040*
4.0K    
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_cascading_standby.log
24K     
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_publisher.log
16K     
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_standby1.log
4.0K    
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_subscriber1.log
12K     
./src/test/recovery/tmp_check/log/regress_log_040_standby_failover_slots_sync

With the patch:

$ du -sh ./src/test/recovery/tmp_check/log/*040*
4.0K    
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_cascading_standby.log
36K     
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_publisher.log
48K     
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_standby1.log
4.0K    
./src/test/recovery/tmp_check/log/040_standby_failover_slots_sync_subscriber1.log
12K     
./src/test/recovery/tmp_check/log/regress_log_040_standby_failover_slots_sync

Regards,

-- 
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com


Reply via email to