From: | Peter Geoghegan <pg(at)heroku(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Oleg Bartunov <obartunov(at)gmail(dot)com> |
Subject: | Re: [COMMITTERS] pgsql: Clean up jsonb code. |
Date: | 2014-05-09 22:23:57 |
Message-ID: | CAM3SWZS04Ff=6TxN81Gf9Pi5YMwXdod+HeQ84Fq+WgAtW_fKpg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Fri, May 9, 2014 at 2:54 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> However, what it looks to me like we've got here is a very bad
> reimplementation of StringInfo buffers. There is for example no
> integer-overflow checking here. Rather than try to bring this code
> up to speed, I think we should rip it out and use StringInfo.
Heikki did specifically consider StringInfo buffers and said they were
not best suited to the task at hand. At the time I thought he meant
that he'd do something domain-specific to avoid unnecessary geometric
growth in the size of the buffer (I like to grow buffers to either
twice their previous size, or just big enough to fit the next thing,
whichever is larger), but that doesn't appear to be the case. Still,
it would be good to know what he meant before proceeding. It probably
had something to do with alignment.
Integer overflow checking probably isn't strictly necessary FWIW,
because there are limits to the size that the buffer can grow to
enforced at various points.
--
Peter Geoghegan
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-05-09 22:24:24 | pgsql: Fix broken allocation logic in recently-rewritten jsonb_util.c. |
Previous Message | Tom Lane | 2014-05-09 21:54:06 | Re: [COMMITTERS] pgsql: Clean up jsonb code. |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-05-09 22:32:29 | Re: [COMMITTERS] pgsql: Clean up jsonb code. |
Previous Message | Tom Lane | 2014-05-09 21:54:06 | Re: [COMMITTERS] pgsql: Clean up jsonb code. |