Re: Dubious code in pg_rewind's process_target_file()

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Dubious code in pg_rewind's process_target_file()
Date: 2020-09-06 15:51:25
Message-ID: 1395091.1599407485@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Heikki Linnakangas <hlinnaka(at)iki(dot)fi> writes:
> On 06/09/2020 18:06, Tom Lane wrote:
>> I propose to remove the lstat() in the back branches, but not touch
>> HEAD so as not to cause extra merge effort for your patch.

> Thanks! Feel free to push it to HEAD, too, the merge conflict will be
> trivial to resolve.

OK, done that way.

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Janes 2020-09-06 16:07:16 moving aggregate bad error message
Previous Message Heikki Linnakangas 2020-09-06 15:13:48 Re: Dubious code in pg_rewind's process_target_file()