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: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: RE: [PATCH]Remove obsolete macro CHECKFLOATVAL in btree_gist
Date: 2021-08-17 11:08:59
Message-ID: OS0PR01MB6113D6869554AFE25E76FB90FBFE9@OS0PR01MB6113.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Friday, August 6, 2021 11:14 PM, tanghy(dot)fnst(at)fujitsu(dot)com <tanghy(dot)fnst(at)fujitsu(dot)com> wrote:
>Commit 6bf0bc842 replaced float.c's CHECKFLOATVAL() macro with static inline subroutines,
>but the fix introduced a performance regression as Tom Lane pointed out and fixed at 607f8ce74.
>
>Found obsolete CHECKFLOATVAL usage in contrib/btree_gist, and tried to fix it according to 607f8ce74.

Added above patch in commit fest:
https://commitfest.postgresql.org/34/3287/

Regards,
Tang

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ranier Vilela 2021-08-17 11:13:35 Re: Fix uninitialized variable access (src/backend/utils/mmgr/freepage.c)
Previous Message Andres Freund 2021-08-17 11:04:27 Re: pgsql: pgstat: Bring up pgstat in BaseInit() to fix uninitialized use o