Re: remove more archiving overhead

From: David Steele <david(at)pgmasters(dot)net>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>, Noah Misch <noah(at)leadboat(dot)com>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, robertmhaas(at)gmail(dot)com, masao(dot)fujii(at)oss(dot)nttdata(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: remove more archiving overhead
Date: 2022-08-02 11:23:35
Message-ID: 9245a010-1f2e-af05-6268-c049934a55d0@pgmasters.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 8/2/22 01:02, Nathan Bossart wrote:
> On Sat, Jul 30, 2022 at 11:51:56PM -0700, Noah Misch wrote:
>> Inviting the administrator to resolve things is more dangerous than just
>> returning true. I recommend making this text more opinionated and simpler:
>> libraries must return true. Alternately, if some library has found a good
>> reason to return false, this paragraph could give the reason. I don't know of
>> such a reason, though.
>
> Your suggestion seems reasonable to me. I've attached a small patch.

+1. I think this is less confusing overall.

Regards,
-David

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Junwang Zhao 2022-08-02 11:27:30 [PATCH] Add a inline function to eliminate duplicate code
Previous Message David Rowley 2022-08-02 11:21:04 Re: Add proper planner support for ORDER BY / DISTINCT aggregates