public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Andrei Lepikhov <[email protected]>
Cc: [email protected]
Cc: Peter Eisentraut <[email protected]>
Subject: Re: Hashed SAOP on composite type with non-hashable column errors at runtime
Date: Mon, 08 Jun 2026 12:00:20 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>

Andrei Lepikhov <[email protected]> writes:
> Now, hash_ok_operator and op_hashjoinable handle all four container-type
> equality operators. Side way is a C extension that lets you create a custom type
> that groups other types marked as HASHES.

Yeah, it's interesting to speculate about what we'd have to do to
allow extensions to invent new kinds of container types.  Right now,
the knowledge of what kinds of containers there are is wired into
a bunch of places.  This fix isn't adding any new places, just fixing
some places whose knowledge was incomplete.  So I'm content with this
for today.

> Fixes in the lookup_type_cache related to the multirange type are also correct
> for me. As well as pg_operator.dat changes.

Thanks for reviewing; I pushed v1-0001 after a bit more
comment-smithing.

>> I'm also unexcited about your 0002 and 0003.  

> I understand about 0003, but what is the problem with 0002?

Let me rephrase that: 0002 is a new feature and hence out of scope
at this point in the development cycle.  If you want to start a
new thread proposing that for v20, go right ahead.

			regards, tom lane

view thread (5+ messages)

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: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Hashed SAOP on composite type with non-hashable column errors at runtime
  In-Reply-To: <[email protected]>

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

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