diff options
| author | Robert Haas | 2024-01-31 15:12:53 +0000 |
|---|---|---|
| committer | Robert Haas | 2024-01-31 15:12:53 +0000 |
| commit | ea18eb7d625573dc369df619f7ff3b9e60e47531 (patch) | |
| tree | d1eb1e98b38b6c21257dc92c40472e97de685f83 /src/backend/backup | |
| parent | d0283387d358e5e0a64a439b7fe906399b59fdd9 (diff) | |
Revise pg_walsummary's 002_blocks test to avoid spurious failures.
Analysis of buildfarm results showed that the code that was intended
to wait for the inserts performed by this test to complete did not
actually do so. Try to make that logic more robust.
Improve error checking elsewhere in the script, too, so that we
don't miss things like poll_query_until failing.
Along the way, fix a bit of pgindent damage introduced by commit
5ddf9973477729cf161b4ad0a1efd52f4fea9c88, which aimed to help us
debug the failures that this commit is trying to fix. It's making
the buildfarm sad.
Discussion: http://postgr.es/m/CA+TgmobWFb8NqyfC31YnKAbZiXf9tLuwmyuvx=iYMXMniPQ4nw@mail.gmail.com
Diffstat (limited to 'src/backend/backup')
| -rw-r--r-- | src/backend/backup/walsummary.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/backup/walsummary.c b/src/backend/backup/walsummary.c index ae314d8b74d..867870aaad7 100644 --- a/src/backend/backup/walsummary.c +++ b/src/backend/backup/walsummary.c @@ -258,7 +258,7 @@ RemoveWalSummaryIfOlderThan(WalSummaryFile *ws, time_t cutoff_time) #else ereport(LOG, (errmsg_internal("removing file \"%s\" cutoff_time=%llu", path, - (unsigned long long) cutoff_time))); + (unsigned long long) cutoff_time))); #endif } |