MIME-Version: 1.0
References: <CAOj6k6d0YO6AO-bhxkfUXPxUi-+YX9-doh2h5D5z0Bm8D2w=OA@mail.gmail.com>
 <CAOVWO5ri14VWdJh+tYt2d5O0VAbZbEhU1C=x-jaxE8qBYgLoUA@mail.gmail.com>
In-Reply-To: <CAOVWO5ri14VWdJh+tYt2d5O0VAbZbEhU1C=x-jaxE8qBYgLoUA@mail.gmail.com>
From: Aditya Gollamudi <adigollamudi@gmail.com>
Date: Fri, 16 Jan 2026 01:35:40 -0800
Message-ID: <CAD-KL_GXFMc-ty9ww94HjdCs-WdbgG2oSLWvnjqztWFBxPq1vA@mail.gmail.com>
Subject: Re: [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs
To: surya poondla <suryapoondla4@gmail.com>
Cc: Dharin Shah <dharinshah95@gmail.com>, pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="0000000000002fa1e406487e11b5"
Archived-At: <https://www.postgresql.org/message-id/CAD-KL_GXFMc-ty9ww94HjdCs-WdbgG2oSLWvnjqztWFBxPq1vA%40mail.gmail.com>
Precedence: bulk

--0000000000002fa1e406487e11b5
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Fri, Jan 9, 2026 at 2:01=E2=80=AFPM surya poondla <suryapoondla4@gmail.c=
om>
wrote:

> Hi Dharin,
>
> I looked at your patch, it looks good.
>
> In the code, I couldn=E2=80=99t find any 2^64 bound on the lexeme count, =
so
> removing that makes sense.
> The added sentence about distinct lexeme count seems to overlap with the
> existing description of tsvector limits, so I=E2=80=99m not sure it adds =
much new
> information.
>
> -Surya Poondla
>

 +1 on this patch, I was also a bit confused on this part of the
documentation. All I could conclude was that the number of lexemes in a
tsvector was limited by existing tsvector limits. I agree with Surya's
comment about the overlap, I think this patch should only remove the line
about the 2^64 bound. Patch applies cleanly!

Adi Gollamudi

--0000000000002fa1e406487e11b5
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Fri, Jan 9, 2026 at 2:01=E2=80=AFPM su=
rya poondla &lt;<a href=3D"mailto:suryapoondla4@gmail.com">suryapoondla4@gm=
ail.com</a>&gt; wrote:</div><div class=3D"gmail_quote gmail_quote_container=
"><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;borde=
r-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div d=
ir=3D"ltr">Hi Dharin,<br><br>I looked at your patch, it looks good.<br><br>=
In the code, I couldn=E2=80=99t find any 2^64 bound on the lexeme count, so=
 removing that makes sense.<br>The added sentence about distinct lexeme cou=
nt seems to overlap with the existing description of tsvector limits, so I=
=E2=80=99m not sure it adds much new information.</div><div class=3D"gmail_=
quote"><div class=3D"gmail_attr"><br>-Surya Poondla</div></div></div></bloc=
kquote><div><br></div><div>=C2=A0+1 on this patch, I was also a bit confuse=
d on this part of the documentation. All I could conclude was that the numb=
er of lexemes in a tsvector was limited by existing tsvector limits. I agre=
e with=C2=A0Surya&#39;s comment about=C2=A0the=C2=A0overlap, I think this p=
atch should only remove the line about the 2^64 bound. Patch applies cleanl=
y!<br><br>Adi Gollamudi</div></div></div>

--0000000000002fa1e406487e11b5--