Horton

Re: Accidental assignment instead of compare in GetOperatorFromCompareType?

Peter Eisentraut <peter@eisentraut.org>

View thread View on postgr.es
From: Peter Eisentraut <peter@eisentraut.org>
To: Jacob Brazeal <jacob.brazeal@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-03-04T08:50:36Z
Lists: pgsql-hackers 
On 03.03.25 06:05, Jacob Brazeal wrote:
> Hi all,
> 
> It looks like commit 630f9a43cece93cb4a5c243b30e34abce6a89514 omitted an 
> equals sign in an ereport() in GetOperatorFromCompareType, giving the line:
> 
> cmptype = COMPARE_EQ ? errmsg("could not identify an equality operator 
> for type %s", format_type_be(opcintype)) :
> 
> I think the impact is just that we'd never see the error messages for 
> the COMPARE_OVERLAP and COMPARE_CONTAINED_BY cases logged.

Fixed, thanks.

Commits

  1. Fix accidental use of = instead of ==

  2. Change gist stratnum function to use CompareType