Skip to content

Allow many (20k+) feeds#4347

Merged
Alkarex merged 4 commits intoFreshRSS:edgefrom
Alkarex:many-feeds
May 9, 2022
Merged

Allow many (20k+) feeds#4347
Alkarex merged 4 commits intoFreshRSS:edgefrom
Alkarex:many-feeds

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented May 5, 2022

Work in progress to allow scaling to 20k+ (testing with 50k+) feeds (one of my own use-cases for a little project at work).
Fix errors such as SQL error listWhereRaw: ERROR: value "42926" is out of range for type smallint
Start of UI and back-end performance improvements accordingly.

Fix errors such as `SQL error listWhereRaw: ERROR: value "42926" is out of range for type smallint`
Start of UI performance improvements accordingly.
@Alkarex Alkarex added this to the 1.20.0 milestone May 5, 2022
@Alkarex Alkarex marked this pull request as draft May 5, 2022 21:12
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 7, 2022
New mode to use only emojis instead of icons.
Considerably improves performances.
Needed for 20k+ feeds scenario FreshRSS#4347
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented May 7, 2022

The needed UI scaling work is done in a distinct PR #4353
On the HTML/CSS side, it looks like this simple UI option (together with the existing option to disable favicons) allows FreshRSS to scale to 50+ feeds.
A bit more work needed on the JavaScript side.

@Alkarex Alkarex marked this pull request as ready for review May 7, 2022 18:06
Alkarex added a commit that referenced this pull request May 9, 2022
New mode to use only emojis instead of icons.
Considerably improves performances.
Needed for 20k+ feeds scenario #4347
@Alkarex Alkarex merged commit 9d1930d into FreshRSS:edge May 9, 2022
@Alkarex Alkarex deleted the many-feeds branch May 9, 2022 13:59
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 10, 2022
New advanced property to automatically simplify the layout when there are many (1k+)  feeds so that FreshRSS works out of the box with 20k+ feeds scenarios FreshRSS#4347

Merge FreshRSS#4356 first.
Alkarex added a commit that referenced this pull request May 15, 2022
* Refactor OPML export categories
Simplify code to comply with types hints.
And renamed a property to plural.

* Automatic simplification of layout for many feeds
New advanced property to automatically simplify the layout when there are many (1k+)  feeds so that FreshRSS works out of the box with 20k+ feeds scenarios #4347

Merge #4356 first.
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 23, 2022
Alkarex added a commit that referenced this pull request May 23, 2022
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented May 23, 2022

#4387

Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 26, 2022
Revert SQLite part of FreshRSS#4387
FreshRSS#4347
Apparently INT and INTEGER are not perfectly synonym in SQLite!
> An INTEGER PRIMARY KEY column is an alias for the rowid, but an INT PRIMARY KEY column is not.
https://www.sqlite.org/stricttables.html#strict_tables
@Alkarex Alkarex mentioned this pull request May 26, 2022
Alkarex added a commit that referenced this pull request May 26, 2022
Revert SQLite part of #4387
#4347
Apparently INT and INTEGER are not perfectly synonym in SQLite!
> An INTEGER PRIMARY KEY column is an alias for the rowid, but an INT PRIMARY KEY column is not.
https://www.sqlite.org/stricttables.html#strict_tables
@Alkarex Alkarex mentioned this pull request Sep 20, 2022
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant