| 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 |
| 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" |