Re: BUG #19633: Unexpected results of IN (subquery) with a non-deterministic collation

From: Andrey Rachitskiy <pl0h0yp1(at)gmail(dot)com>
To: Tender Wang <tndrwang(at)gmail(dot)com>
Cc: syzhong16(at)gmail(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org, Alexander Korotkov <aekorotkov(at)gmail(dot)com>
Subject: Re: BUG #19633: Unexpected results of IN (subquery) with a non-deterministic collation
Date: 2026-08-21 05:01:59
Message-ID: CAB8bMivfsjkq_kG3VehogS8-PNMMxuVdprpXKMPxoEyh6WS-Rw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

пт, 21 авг. 2026 г. в 06:54, Tender Wang <tndrwang(at)gmail(dot)com>:

> The approach looks good to me. I'd suggest adjusting the comment as
> follows:
> ...
> /* so far so good, keep building lists */
> semi_operators = lappend_oid(semi_operators, opno);
>
> /*
> * Ensure that the RHS expression exposes the join operator's input
> * collation. The expression will later be used as a grouping key when
> * unique-ifying the RHS, so its collation must agree with the semijoin
> * equality semantics.
> */
> semi_rhs_exprs =
> lappend(semi_rhs_exprs,
> canonicalize_ec_expression((Expr *) copyObject(right_expr),
> exprType(right_expr),
> op->inputcollid));
>
> Dear Tender,

Thanks for the review.
Made the changes - v2 attached.

--
Regards,
Rachitskiy Andrey

Attachment Content-Type Size
v2-0001-Fix-semijoin-RHS-unique-ification-to-use-join-collation.patch text/x-patch 4.6 KB

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Ayush Tiwari 2026-08-21 05:18:30 Re: BUG #19629: pg_restore_relation_stats reports XX000 instead of proper SQLSTATE for input validation errors
Previous Message PG Bug reporting form 2026-08-21 03:44:42 BUG #19634: Hash partition with large MODULUS causes "invalid memory alloc request size"