From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
Cc: | Erik Wienhold <ewie(at)ewie(dot)name>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Underscore in positional parameters? |
Date: | 2024-05-15 23:11:11 |
Message-ID: | ZkVBD_y7gWnQQWEV@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, May 15, 2024 at 01:59:36PM +0200, Peter Eisentraut wrote:
> On 14.05.24 18:07, Erik Wienhold wrote:
>> Patch 0002 replaces atol with pg_strtoint32_safe in the backend parser
>> and strtoint in ECPG. This fixes overflows like:
>
> Seems like a good idea, but as was said, this is an older issue, so let's
> look at that separately.
Hmm, yeah. I would be really tempted to fix that now.
Now, it has been this way for ages, and with my RMT hat on (aka I need
to show the example), I'd suggest to wait for when the v18 branch
opens as there is no urgency. I'm OK to apply it myself at the end,
the patch is a good idea.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2024-05-15 23:17:17 | Re: Log details for stats dropped more than once |
Previous Message | Michael Paquier | 2024-05-15 23:07:11 | Re: Introduce new multi insert Table AM and improve performance of various SQL commands with it for Heap AM |