| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Making aggregate deserialization (and WAL receive) functions slightly faster |
| Date: | 2023-10-17 07:39:52 |
| Message-ID: | CAApHDvps5q4+CoJawcPr=jm0WmsJRWTRZXMS8UVcrepcwYAG3Q@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, 16 Oct 2023 at 05:56, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> * in initStringInfoFromString, str->maxlen must be set to len+1 not len
>
> * comment in exec_bind_message doesn't look like pgindent will like it
>
> * same in record_recv, plus it has a misspelling "Initalize"
>
> * in stringinfo.c, inclusion of pg_bitutils.h seems no longer needed
Thank you for looking again. I've addressed all of these in the attached.
> I guess the next question is whether we want to stop here or
> try to relax the requirement about NUL-termination. I'd be inclined
> to call that a separate issue deserving a separate commit, so maybe
> we should go ahead and commit this much anyway.
I am keen to see this relaxed. I agree that a separate effort is best.
David
| Attachment | Content-Type | Size |
|---|---|---|
| initstringinfo_changes_v4.patch | text/plain | 10.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dmitry Dolgov | 2023-10-17 08:15:41 | Re: pg_stat_statements and "IN" conditions |
| Previous Message | Ashutosh Bapat | 2023-10-17 07:25:46 | serial and partitioned table |