Does this conditional branch have any special meaning？

[zxbzswxr]

pg_probackup/src/merge.c

Lines 339 to 341 in 7792b98

	if ((full_backup->status == BACKUP_STATUS_MERGED ||
	full_backup->status == BACKUP_STATUS_MERGED) &&
	dest_backup->start_time != full_backup->merge_dest_backup)

what if we rewrite it as
if(full_backup->status == BACKUP_STATUS_MERGED && dest_backup->start_time != full_backup->merge_dest_backup)
......

[fukanchik]

Thanks, for reporting this! We suspect it should be MERGED+MERGING. Will be fixed in one of the next releases.

[zxbzswxr]

Thanks, for reporting this! We suspect it should be MERGED+MERGING. Will be fixed in one of the next releases.

so, it will be
if ((full_backup->status == BACKUP_STATUS_MERGED ||
full_backup->status == BACKUP_STATUS_MERGING) &&
dest_backup->start_time != full_backup->merge_dest_backup)
..... ?
that's all ??

[fukanchik]

Yes

[zxbzswxr]

Yes

Thank you.