MIME-Version: 1.0
References: 
 <CAOj6k6d0YO6AO-bhxkfUXPxUi-+YX9-doh2h5D5z0Bm8D2w=OA@mail.gmail.com>
 <CAOVWO5ri14VWdJh+tYt2d5O0VAbZbEhU1C=x-jaxE8qBYgLoUA@mail.gmail.com>
 <CAD-KL_GXFMc-ty9ww94HjdCs-WdbgG2oSLWvnjqztWFBxPq1vA@mail.gmail.com>
 <1048391.1774562546@sss.pgh.pa.us>
In-Reply-To: <1048391.1774562546@sss.pgh.pa.us>
From: Dharin Shah <dharinshah95@gmail.com>
Date: Thu, 26 Mar 2026 23:48:18 +0100
Message-ID: 
 <CAOj6k6esm7NoX8sGDd7FVJ=f+ScXqmjGSiwQu0MBW0-amvTxWA@mail.gmail.com>
Subject: Re: [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Aditya Gollamudi <adigollamudi@gmail.com>,
 surya poondla <suryapoondla4@gmail.com>,
	pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="000000000000eb4256064df52e01"
Archived-At: 
 <https://www.postgresql.org/message-id/CAOj6k6esm7NoX8sGDd7FVJ%3Df%2BScXqmjGSiwQu0MBW0-amvTxWA%40mail.gmail.com>
Precedence: bulk

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

Sorry to drop the ball on this, will get back to cleaning it up and create
a new patch

Thanks,
Dharin

On Thu, Mar 26, 2026 at 11:02=E2=80=AFPM Tom Lane <tgl@sss.pgh.pa.us> wrote=
:

> Aditya Gollamudi <adigollamudi@gmail.com> writes:
> >  +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 li=
ne
> > about the 2^64 bound. Patch applies cleanly!
>
> What I'm inclined to do is just drop that <listitem> entirely,
> since it's implied by the 1MB space limit.  The patch as-submitted
> accomplishes about the same thing, but takes more words to do it.
>
>                         regards, tom lane
>

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

<div dir=3D"ltr">Sorry to drop the ball on this, will get back to cleaning =
it up and create a new patch<br><br>Thanks,<br>Dharin</div><br><div class=
=3D"gmail_quote gmail_quote_container"><div dir=3D"ltr" class=3D"gmail_attr=
">On Thu, Mar 26, 2026 at 11:02=E2=80=AFPM Tom Lane &lt;<a href=3D"mailto:t=
gl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&gt; wrote:<br></div><blockquote cla=
ss=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid =
rgb(204,204,204);padding-left:1ex">Aditya Gollamudi &lt;<a href=3D"mailto:a=
digollamudi@gmail.com" target=3D"_blank">adigollamudi@gmail.com</a>&gt; wri=
tes:<br>
&gt;=C2=A0 +1 on this patch, I was also a bit confused on this part of the<=
br>
&gt; documentation. All I could conclude was that the number of lexemes in =
a<br>
&gt; tsvector was limited by existing tsvector limits. I agree with Surya&#=
39;s<br>
&gt; comment about the overlap, I think this patch should only remove the l=
ine<br>
&gt; about the 2^64 bound. Patch applies cleanly!<br>
<br>
What I&#39;m inclined to do is just drop that &lt;listitem&gt; entirely,<br=
>
since it&#39;s implied by the 1MB space limit.=C2=A0 The patch as-submitted=
<br>
accomplishes about the same thing, but takes more words to do it.<br>
<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 regards, tom lane<br>
</blockquote></div>

--000000000000eb4256064df52e01--