Skip to content

WebUI: Improve table performance#23752

Merged
Chocobo1 merged 3 commits intoqbittorrent:masterfrom
Piccirello:tom/table-performance
Feb 1, 2026
Merged

WebUI: Improve table performance#23752
Chocobo1 merged 3 commits intoqbittorrent:masterfrom
Piccirello:tom/table-performance

Conversation

@Piccirello
Copy link
Member

Switch from an array to a set for O(n) operations.

@Piccirello Piccirello added this to the 5.2 milestone Jan 17, 2026
@Piccirello Piccirello added the WebUI WebUI-related issues/changes label Jan 17, 2026
This provides constant time lookups.
@Piccirello Piccirello force-pushed the tom/table-performance branch from 000297c to 0d23d10 Compare January 19, 2026 19:07
@Piccirello Piccirello requested a review from a team January 19, 2026 19:07
@Piccirello Piccirello changed the title WebUI: Minor table performance improvements WebUI: Table performance improvements Jan 19, 2026
@glassez glassez changed the title WebUI: Table performance improvements WebUI: Improve table performance Jan 24, 2026
This introduces a new `setRows` method for efficiently updating the rows of an existing table. The caller provides `setRows` will the full set of rows to display, and `setRows` efficiently updates existing rows and deletes unused rows. `clear()` is now effectively only called when the full data set changes (i.e. a different torrent is selected).
@Piccirello Piccirello force-pushed the tom/table-performance branch from 0d23d10 to 4055016 Compare January 26, 2026 01:33
@Chocobo1 Chocobo1 merged commit 1821cc4 into qbittorrent:master Feb 1, 2026
15 checks passed
@Chocobo1
Copy link
Member

Chocobo1 commented Feb 1, 2026

@Piccirello
Thank you!

@Chocobo1
Copy link
Member

Chocobo1 commented Feb 3, 2026

@Piccirello
Using RSS in WebUI seems to be spamming error messages in browser console since this PR. Not sure whether other functionalities are affected.

Chocobo1 added a commit to Chocobo1/qBittorrent that referenced this pull request Feb 6, 2026
This change is incomplete and causes RSS to spam errors in browser
console.
Fix up: qbittorrent#23752.
@Piccirello
Copy link
Member Author

@Piccirello Using RSS in WebUI seems to be spamming error messages in browser console since this PR. Not sure whether other functionalities are affected.

Will look into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Performance WebUI WebUI-related issues/changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants