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