iTunes and Rhythmbox feature are far too slow
Bug #528264 reported by
RJ Skerry-Ryan
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mixxx |
Fix Released
|
High
|
RAFFI TEA | ||
| 1.8 |
Won't Fix
|
High
|
RAFFI TEA | ||
| 1.9 |
Won't Fix
|
High
|
RAFFI TEA | ||
Bug Description
Rhythmbox and iTunes load the libraries into memory and do not have indexes. Not only is this bad for memory usage, but it also results in very slow library performance when you load the library, switch the view, or sort a column.
What we should do is create temporary SQL tables for both tracks and playlists.
Unfortunately, I almost consider this blocking for both the Rhythmbox and iTunes features. Any medium-sized library can negatively affect Mixxx's performance.
Related branches
lp:~mixxxdevelopers/mixxx/features_rhythmbox
- RJ Skerry-Ryan: Approve
-
Diff: 4495 lines (+1341/-1470)38 files modifiedmixxx/build/depends.py (+19/-12)
mixxx/res/schema.xml (+29/-0)
mixxx/src/library/abstractxmltrackmodel.cpp (+0/-218)
mixxx/src/library/abstractxmltrackmodel.h (+0/-100)
mixxx/src/library/autodjfeature.cpp (+1/-1)
mixxx/src/library/basesqltablemodel.cpp (+13/-2)
mixxx/src/library/basesqltablemodel.h (+3/-0)
mixxx/src/library/cratefeature.h (+1/-1)
mixxx/src/library/itunes/itunesfeature.cpp (+107/-59)
mixxx/src/library/itunes/itunesfeature.h (+20/-11)
mixxx/src/library/itunes/itunesplaylistmodel.cpp (+3/-2)
mixxx/src/library/itunes/itunesplaylistmodel.h (+1/-1)
mixxx/src/library/itunes/itunestrackmodel.cpp (+3/-2)
mixxx/src/library/itunes/itunestrackmodel.h (+1/-1)
mixxx/src/library/library.cpp (+6/-4)
mixxx/src/library/libraryfeature.cpp (+1/-0)
mixxx/src/library/libraryfeature.h (+4/-0)
mixxx/src/library/mixxxlibraryfeature.cpp (+1/-1)
mixxx/src/library/promotracksfeature.cpp (+0/-1)
mixxx/src/library/proxytrackmodel.cpp (+0/-130)
mixxx/src/library/proxytrackmodel.h (+0/-53)
mixxx/src/library/rhythmbox/rhythmboxfeature.cpp (+365/-50)
mixxx/src/library/rhythmbox/rhythmboxfeature.h (+38/-12)
mixxx/src/library/rhythmbox/rhythmboxplaylistmodel.cpp (+169/-254)
mixxx/src/library/rhythmbox/rhythmboxplaylistmodel.h (+32/-74)
mixxx/src/library/rhythmbox/rhythmboxtrackmodel.cpp (+127/-167)
mixxx/src/library/rhythmbox/rhythmboxtrackmodel.h (+41/-44)
mixxx/src/library/sidebarmodel.cpp (+123/-100)
mixxx/src/library/sidebarmodel.h (+8/-1)
mixxx/src/library/trackcollection.cpp (+1/-1)
mixxx/src/library/traktor/traktorfeature.cpp (+141/-101)
mixxx/src/library/traktor/traktorfeature.h (+18/-10)
mixxx/src/library/traktor/traktorplaylistmodel.cpp (+25/-28)
mixxx/src/library/traktor/traktorplaylistmodel.h (+7/-7)
mixxx/src/library/traktor/traktortablemodel.cpp (+17/-16)
mixxx/src/library/traktor/traktortablemodel.h (+7/-6)
mixxx/src/widget/wlibrarysidebar.cpp (+7/-0)
mixxx/src/widget/wlibrarysidebar.h (+2/-0)
| description: | updated |
| Changed in mixxx: | |
| milestone: | none → 1.8.0 |
| status: | New → Triaged |
| importance: | Undecided → High |
Revision history for this message
| RJ Skerry-Ryan (rryan) wrote | #1 |
| Changed in mixxx: | |
| milestone: | 1.8.0 → 1.8.1 |
| Changed in mixxx: | |
| status: | Triaged → Confirmed |
Revision history for this message
| RAFFI TEA (raffitea) wrote | #2 |
| Changed in mixxx: | |
| milestone: | 1.8.1 → 1.8.2 |
Revision history for this message
| RAFFI TEA (raffitea) wrote | #3 |
| Changed in mixxx: | |
| assignee: | nobody → RAFFI TEA (raffitea) |
| Changed in mixxx: | |
| status: | Confirmed → In Progress |
| Changed in mixxx: | |
| milestone: | 1.8.2 → none |
Revision history for this message
| RAFFI TEA (raffitea) wrote | #4 |
| Changed in mixxx: | |
| milestone: | none → 1.10.0 |
| status: | In Progress → Fix Committed |
| Changed in mixxx: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Swiftb0y (swiftb0y) wrote | #5 |
| lock status: | Metadata changes locked and limited to project staff |
To post a comment you must log in.
We'll fix this is 1.8.1