RE: [PATCH]Remove obsolete macro CHECKFLOATVAL in btree_gist

From: "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: RE: [PATCH]Remove obsolete macro CHECKFLOATVAL in btree_gist
Date: 2021-08-19 05:10:07
Message-ID: OS0PR01MB61139DD9B392ECBF7F972D41FBC09@OS0PR01MB6113.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thursday, August 19, 2021 12:28 PM, Michael Paquier <michael(at)paquier(dot)xyz> wrote
>On Wed, Aug 18, 2021 at 10:04:04AM +0900, Michael Paquier wrote:
>> Yes, that does not seem wise on performance grounds. The case of
>> !zero_is_valid is never reached, so it seems like this code was just a
>> copy-paste from the float code in the backend. Your patch looks right
>> to me.
>
>Applied.

Thanks for you check and commit, I've changed the patch's commit fest status to 'committed'.
https://commitfest.postgresql.org/34/3287/

Regards,
Tang

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Nancarrow 2021-08-19 05:18:00 Re: Skipping logical replication transactions on subscriber side
Previous Message Amit Kapila 2021-08-19 05:06:01 Re: Skipping logical replication transactions on subscriber side