Re: pgsql: Make cancel request keys longer

2025-05-23 Thread Jacob Champion
On Thu, May 8, 2025 at 11:38 PM Heikki Linnakangas wrote: > It didn't occur to me that you could write it simply as 'msgLength - 4'. > That depends on knowing that the preceding fields are exactly 4 bytes > long, but that's clear enough if we just add a comment on that, see > attached. Sorry for

Re: pgsql: Make cancel request keys longer

2025-05-08 Thread Heikki Linnakangas
On 09/05/2025 01:28, Jacob Champion wrote: On Thu, May 8, 2025 at 12:11 PM Heikki Linnakangas wrote: Polished this up a tiny bit, and committed. Thanks! I think the uint8->int change for cancel_key_len is more than just cosmetic; it most likely fixes a bug where a key size of 256 wrapped arou

Re: pgsql: Make cancel request keys longer

2025-05-08 Thread Jacob Champion
On Thu, May 8, 2025 at 12:11 PM Heikki Linnakangas wrote: > Polished this up a tiny bit, and committed. Thanks! I think the uint8->int change for cancel_key_len is more than just cosmetic; it most likely fixes a bug where a key size of 256 wrapped around to 0. I'll double-check that this fixes th

Re: pgsql: Make cancel request keys longer

2025-05-08 Thread Heikki Linnakangas
On 09/04/2025 13:46, Heikki Linnakangas wrote: On 09/04/2025 13:28, Heikki Linnakangas wrote: On 09/04/2025 12:39, Peter Eisentraut wrote: On 09.04.25 10:53, Heikki Linnakangas wrote: On 08/04/2025 22:41, Heikki Linnakangas wrote: On 08/04/2025 20:06, Peter Eisentraut wrote: While I was looki

Re: pgsql: Make cancel request keys longer

2025-04-09 Thread Heikki Linnakangas
(moving to pgsql-hackers) On 09/04/2025 12:39, Peter Eisentraut wrote: On 09.04.25 10:53, Heikki Linnakangas wrote: On 08/04/2025 22:41, Heikki Linnakangas wrote: On 08/04/2025 20:06, Peter Eisentraut wrote: While I was looking at this, I suggest to make the first argument void *.  This is co

Re: pgsql: Make cancel request keys longer

2025-04-09 Thread Heikki Linnakangas
On 09/04/2025 13:28, Heikki Linnakangas wrote: On 09/04/2025 12:39, Peter Eisentraut wrote: On 09.04.25 10:53, Heikki Linnakangas wrote: On 08/04/2025 22:41, Heikki Linnakangas wrote: On 08/04/2025 20:06, Peter Eisentraut wrote: While I was looking at this, I suggest to make the first argument