this post was submitted on 06 Aug 2023
1444 points (99.0% liked)
Announcements
23319 readers
1 users here now
Official announcements from the Lemmy project. Subscribe to this community or add it to your RSS reader in order to be notified about new releases and important updates.
You can also find major news on join-lemmy.org
founded 5 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Maybe I'm completely misremembering things, but at some point wasn't there a hotfix to Lemmy that hard-limited how many comments a thread could have? Does anyone know if there's a maximum and if so how many?
Just wondering, cause uh, I could see this one having a lot of comments.
The fix you are referring to only limits how many comments can be retrieved in a single API call (300). This limit is only used when specific parameters are passed, not in all cases.
Tree-paging is a pretty complicated issue, and we really do need some DB / SQL experts to help us with figuring out how to page them correctly. The limit is 300, but only for the top-level comment fetch, which could also have different slices whether you sort by top, or new, and doesn't apply to the nested comments, which could have thousands.
The limit is a kludge, because ppl were creating thousands of comments, and without proper paging, it was affecting performance.