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

limit feeds and delete materialized views #2469

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

unixfox
Copy link
Member

@unixfox unixfox commented Oct 6, 2021

Recreated from #2058 and rebased.

@unixfox unixfox marked this pull request as ready for review October 19, 2021 06:55
@unixfox unixfox requested a review from a team as a code owner October 19, 2021 06:55
@unixfox unixfox requested review from syeopite and removed request for a team October 19, 2021 06:55
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from eb086a3 to 80e284d Compare October 19, 2021 07:12
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 19d49cd to 71f11be Compare October 27, 2021 07:33
@syeopite syeopite added the blocked require something else first label Oct 27, 2021
@syeopite
Copy link
Member

This is blocked until we can get an automatic migration system in Invidious.

@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 71f11be to 2245206 Compare December 15, 2021 18:29
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 2245206 to 02de77a Compare January 5, 2022 13:17
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 02de77a to f067670 Compare January 25, 2022 20:07
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from f067670 to 590e9bf Compare February 2, 2022 08:49
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 590e9bf to d73140a Compare February 15, 2022 12:53
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch 2 times, most recently from 477a778 to cf51c5e Compare February 25, 2022 08:48
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch 2 times, most recently from bd2cfd7 to 6a19f66 Compare April 9, 2022 20:18
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch from 6a19f66 to a6056f6 Compare June 23, 2023 19:48
@unixfox unixfox force-pushed the limit-feeds-materialized-views branch 2 times, most recently from 3befb12 to 1e6a035 Compare July 26, 2023 13:58
@999eagle
Copy link

fwiw I'm currently testing this PR on my Invidious instance (https://iv.catgirl.cloud) with the database index created manually. It's reducing the write load on my database a lot as some of the larger views are refreshed and thus rewritten very often. So once automatic migrations are done I hope this PR can be revisited!

@Fijxu
Copy link
Contributor

Fijxu commented Mar 21, 2024

Applied this commit on my instance 12 hours ago. Works well and this is really really useful for selfhosted instances like mine. Invidious was literally killing my SSD writing a lot of GiB per hour, now it's just normal.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked require something else first
Projects
Status: Planned
Development

Successfully merging this pull request may close these issues.

4 participants