Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[performance/chore] Update local + federated (public) timelines to use internal/timeline #2842

Open
tsmethurst opened this issue Apr 16, 2024 · 0 comments
Labels
chore a pain in the butt that needs doing performance

Comments

@tsmethurst
Copy link
Contributor

Currently when a user requests the local / public timeline via API, we go straight to making database calls and filtering items out that don't apply.

This works more or less OK, but can cause performance issues (lots of database calls) and issues where all items get filtered out and the user gets an empty timeline back (#2410, #2273).

We mostly mitigated this in #2784, but the issue still crops up again occasionally.

If we were to store new statuses eligible for public / local timelining as they come in, using a timeline from the internal/timeline package, we could speed up serving timelines, and also make it easier to avoid filtering out every entry.

@tsmethurst tsmethurst added performance chore a pain in the butt that needs doing labels Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore a pain in the butt that needs doing performance
Projects
None yet
Development

No branches or pull requests

1 participant