Re: pgsql: Add pg_plan_advice contrib module.

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Robert Haas <rhaas(at)postgresql(dot)org>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add pg_plan_advice contrib module.
Date: 2026-03-16 21:11:54
Message-ID: CAApHDvquH6wnp4fhpaCOkC4R3KAvr2BOTbhhDPDQCBNR3YbLMQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Fri, 13 Mar 2026 at 06:01, Robert Haas <rhaas(at)postgresql(dot)org> wrote:
> Add pg_plan_advice contrib module.

Hi Robert,

I was just building with VS2026 when I noticed the following warning:

../contrib/pg_plan_advice/pgpa_planner.c(1195): warning C4319: '~':
zero extending 'uint32' to 'uint64' of greater size

I guess the compiler is concerned that you might not have meant to
unconditionally switch off all the upper 32-bits of the 64-bit
variable. Adding the following does get rid of the warning for me,
but not sure if that's what you had intended. I've not studied the
code enough to understand why there are two different integer widths
being used for the bits.

- *pgs_mask_p &= ~(PGS_JOIN_ANY & ~join_mask);
+ *pgs_mask_p &= (uint32) ~(PGS_JOIN_ANY & ~join_mask);

David

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Fujii Masao 2026-03-16 23:13:15 pgsql: Fix WAL flush LSN used by logical walsender during shutdown
Previous Message Jeff Davis 2026-03-16 20:44:22 pgsql: Clean up postgres_fdw/t/010_subscription.pl.

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2026-03-16 21:15:45 Re: More speedups for tuple deformation
Previous Message Masahiko Sawada 2026-03-16 21:03:39 Re: Patch for migration of the pg_commit_ts directory