Re: Collation & ctype method table, and extension hooks

From: Noah Misch <noah(at)leadboat(dot)com>
To: Jeff Davis <pgsql(at)j-davis(dot)com>
Cc: Andreas Karlsson <andreas(at)proxel(dot)se>, pgsql-hackers(at)postgresql(dot)org, Peter Eisentraut <peter(at)eisentraut(dot)org>
Subject: Re: Collation & ctype method table, and extension hooks
Date: 2026-07-07 00:11:14
Message-ID: 20260707001114.99@rfd.leadboat.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jul 06, 2026 at 02:31:06PM -0700, Jeff Davis wrote:
> On Mon, 2026-06-29 at 18:29 -0700, Noah Misch wrote:
> >   I've copied Claude's whole
> > review for the commit to
> > https://docs.google.com/document/d/19_xKpKUQlY5N51qbfXyipXs3rqIYEC3qk5oaUDJ48YA/edit?tab=t.0
> > .
> > Two of its other findings sound potentially important, but I didn't
> > verify
> > them.
>
> I'll refer to the issue numbers below (NB: the doc's summary uses
> different numbering, which is confusing):
>
> Issue 1: Real issue for windows and code points > 0xFFFF. Other cases
> are just defensive, I believe. Fixed in 0003.
>
> Issue 2: I don't see an issue here, though sort of related to the fix
> in 0003.
>
> Issue 3: I don't see an issue here.
>
> Issue 4: The issue you raised above; fixed in 0001.
>
> Issue 5: I don't see an issue here.
>
> I also addressed summary item 4, the comment block fix, in 0002. And I
> added some missing casts in 0004.
>
> Does that cover the two other findings you were concerned about?

Yep. I didn't read the patches, but this per-issue triage is reasonable.

> In the future, if practical, can you please include reports like this
> as text attachments for the sake of the archives?

I've attached it.

Attachment Content-Type Size
5a38104-opus4.7extra.md text/plain 12.0 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Richard Guo 2026-07-07 00:46:34 Re: Fix HAVING-to-WHERE pushdown with mismatched operator families
Previous Message Amit Langote 2026-07-06 23:52:03 Re: JSON_VALUE/JSON_TABLE DEFAULT expression ignores RETURNING typmod