Re: BUG #18080: to_tsvector fails for long text input
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: uwe.binder@pass-consulting.com, pgsql-bugs@lists.postgresql.org
Date: 2023-09-22T19:31:10Z
Lists: pgsql-bugs
Attachments
- pack-ParsedWord-tightly.patch (text/x-diff) patch
I wrote:
> BTW, the field order in ParsedWord is such that there's a fair
> amount of wasted pad space on 64-bit builds. I doubt we can
> get away with rearranging it in released branches; but maybe
> it's worth doing something about that in HEAD, to push out
> the point at which you hit the 1Gb limit.
I poked at that a little bit. We can reduce 64-bit sizeof(ParsedWord)
from 40 bytes to 24 bytes with the attached patch. The main thing
needed to make this pack tightly is to reduce the "alen" field from
uint32 to uint16. While it's not immediately obvious that that's
a good thing to do, a look at the one place where alen is increased
(uniqueWORD() in to_tsany.c) shows that it cannot get to more than
twice MAXNUMPOS:
if (res->pos.apos[0] < MAXNUMPOS - 1 && ...)
{
if (res->pos.apos[0] + 1 >= res->alen)
{
res->alen *= 2;
res->pos.apos = (uint16 *) repalloc(res->pos.apos, sizeof(uint16) * res->alen);
}
MAXNUMPOS is currently 256, and even if it's possible to increase
that it seems unlikely that we'd want to make it more than 32k.
So this limitation seems OK to me.
regards, tom lane
Commits
-
Pack struct ParsedWord more tightly.
- 036297cf1b84 17.0 landed
-
Limit to_tsvector_byid's initial array allocation to something sane.
- cf1c65070a89 17.0 landed
- a2799f53d519 14.10 landed
- 8465efc1a0e7 16.1 landed
- 71bb73f60be7 15.5 landed
- 52a676ef4fc8 13.13 landed
- 4a81ed29d3c9 12.17 landed
- 0fb91ed2b0af 11.22 landed