Re: [PG19]pg_verifybackup never finishes on a gzip-compressed tar backup

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: shihao zhong <zhong950419(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, sulamul(at)gmail(dot)com
Subject: Re: [PG19]pg_verifybackup never finishes on a gzip-compressed tar backup
Date: 2026-10-08 14:14:54
Message-ID: 59d54b10-ceb5-41b4-a04b-c7d844c7d177@dunslane.net
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 2026-10-07 We 8:48 PM, Chao Li wrote:
>
>> On Oct 7, 2026, at 19:44, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>>
>>> On 6 Oct 2026, at 03:48, shihao zhong <zhong950419(at)gmail(dot)com> wrote:
>>> I took a look at the code and I think we are solving different problems.
>> I concur that this is a separate issue. Today I returned to looking at the
>> truncation patchset and I am including this fix in that review effort.
>>
>> --
>> Daniel Gustafsson
>>
> Sorry for the confusion. I was on vacation and didn’t look into the patch closely.
>
> This seems to be an old bug that couldn’t be triggered before PG19 (b15c1513984 added tar support to pg_waldump). So, I agree with Melanie that this should be an 19 open item.
>
> The patch calls inflateReset() after each member finishes, allowing decompression to continue with the next member, which looks like a correct fix to me.
>

Yes, looks good to me. I will commit later today.

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Burd 2026-10-08 14:25:34 Re: Let an ordering index scan hand its ORDER BY value to the target list
Previous Message Raja Sai pranav 2026-10-08 14:09:22 Re: Add a hint to the "WAL summaries are required" errors