pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Disallow creating binary-coercible casts involving range types.
Date: Wed, 21 Aug 2024 16:00:23 +0000
Message-ID: <E1sgnl1-000nCX-CM@gemulon.postgresql.org> (raw)

Disallow creating binary-coercible casts involving range types.

For a long time we have forbidden binary-coercible casts to or from
composite and array types, because such a cast cannot work correctly:
the type OID embedded in the value would need to change, but it won't
in a binary coercion.  That reasoning applies equally to range types,
but we overlooked installing a similar restriction here when we
invented range types.  Do so now.

Given the lack of field complaints, we won't change this in stable
branches, but it seems not too late for v17.

Per discussion of a problem noted by Peter Eisentraut.

Discussion: https://postgr.es/m/076968e1-0852-40a9-bc0b-117cd3f0e43c@eisentraut.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/86488cdf1249f86cff75c2446f670be49cb55055

Modified Files
--------------
src/backend/commands/functioncmds.c | 35 ++++++++++++++++++++++++-----------
1 file changed, 24 insertions(+), 11 deletions(-)



view thread (2+ messages)  latest in thread

Message-ID: <E1sgnl1-000nCX-CM@gemulon.postgresql.org>
Permalink:  ../E1sgnl1-000nCX-CM@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1sgnl1-000nCX-CM@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Disallow creating binary-coercible casts involving range types.
  In-Reply-To: <E1sgnl1-000nCX-CM@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox