public inbox for [email protected]  

help / color / mirror / Atom feed

[PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs
2+ messages / 1 participants
[nested] [flat]

* [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs
@ 2025-12-27 21:09 Dharin Shah <[email protected]>
  2026-03-26 22:48 ` Re: [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs Dharin Shah <[email protected]>
  0 siblings, 1 reply; 2+ messages in thread

From: Dharin Shah @ 2025-12-27 21:09 UTC (permalink / raw)
  To: [email protected]

--000000000000cd42f80646f56cbd
Content-Type: multipart/alternative; boundary="000000000000cd42f60646f56cbb"

--000000000000cd42f60646f56cbb
Content-Type: text/plain; charset="UTF-8"

Hello,

A minor doc patch for this page
https://www.postgresql.org/docs/current/textsearch-limitations.html
and this line

*- The number of lexemes must be less than 2^64*

Docs wrongly claim "lexemes must be < 2^64" but the actual constraint is
1 MB total storage (MAXSTRPOS), and no 2^64 check exists in the code.



^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* Re: [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs
  2025-12-27 21:09 [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs Dharin Shah <[email protected]>
@ 2026-03-26 22:48 ` Dharin Shah <[email protected]>
  0 siblings, 0 replies; 2+ messages in thread

From: Dharin Shah @ 2026-03-26 22:48 UTC (permalink / raw)
  To: Tom Lane <[email protected]>; +Cc: Aditya Gollamudi <[email protected]>; surya poondla <[email protected]>; [email protected]

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 PM Tom Lane <[email protected]> wrote:

> Aditya Gollamudi <[email protected]> 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 line
> > 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
>


^ permalink  raw  reply  [nested|flat] 2+ messages in thread

end of thread, other threads:[~2026-03-26 22:48 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2025-12-27 21:09 [PATCH][DOC][MINOR] Fix incorrect lexeme limit in textsearch docs Dharin Shah <[email protected]>
2026-03-26 22:48 ` Dharin Shah <[email protected]>

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox