After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 541469 - UI often hangs when changing tracks while cover art is fetching
UI often hangs when changing tracks while cover art is fetching
Status: RESOLVED WONTFIX
Product: banshee
Classification: Other
Component: Other Extensions
git master
Other Linux
: Normal major
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
gnome[unmaintained]
: 540526 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-07-03 22:49 UTC by Sandy Armstrong
Modified: 2020-03-17 08:24 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Sandy Armstrong 2008-07-03 22:49:45 UTC
Steps to reproduce:

0. Enable cover art fetching.
1. Begin playing a track.
2. While cover art extension indicates that it's still doing something, skip to next track.

Expected result:

Next track starts playing immediately.

Actual result:

Banshee UI hangs for awhile, old track keeps playing.  After many seconds, next track finally begins to play.


I can't quite nail this down...sometimes it works fine, but with the majority of my tracks it hangs.  It seems to be related to whether or not they already have cover art.  Disabling the cover art extension makes the problem go away.
Comment 1 Michał Sawicz 2008-07-13 19:37:46 UTC
Yeah I had to disable the fetcher, too, banshee hanged almost every time song changed (the fetcher seemed to hang earlier, too).
Comment 2 Bertrand Lorentz 2008-07-20 19:36:16 UTC
I think the source of the problem is the SQL query used in the beginning of CoverArtJob.Run(). It takes 2s of CPU time to run on my system.

It's used on every track change to check if it should try to fetch some cover art.
Comment 3 Bertrand Lorentz 2008-07-22 19:11:41 UTC
*** Bug 540526 has been marked as a duplicate of this bug. ***
Comment 4 Gabriel Burt 2008-07-25 20:30:55 UTC
Snorp committed a fix so that the cover art refresh isn't triggered by the play/skip count getting incremented or rating changed, and I committed two fixes, one to set the job as lowest priority, and another fixing a big performance regression I introduced ~ a week ago in a SQL query.  Fixed!
Comment 5 Sandy Armstrong 2008-07-25 20:51:42 UTC
1. Tools->Cover Art->Download Cover Art
2. Try to change to next track

Hang.
Comment 6 Sandy Armstrong 2008-07-25 20:52:11 UTC
That being said, snorp's fix makes this bug much less likely to show up.
Comment 7 Sandy Armstrong 2008-07-25 21:32:59 UTC
(In reply to comment #5)
> 1. Tools->Cover Art->Download Cover Art
> 2. Try to change to next track while fetching occurs
> 
> Hang.

Can anybody else reproduce this with r4254 or later?
Comment 8 Andrew Conkling 2008-07-26 02:51:23 UTC
(In reply to comment #7)
> Can anybody else reproduce this with r4254 or later?

No. Even if I clear my album-art cache, the cover art fetcher is too damned fast. :P I don't even get any feedback that it's downloading anything.
Comment 9 Sandy Armstrong 2008-07-26 04:11:29 UTC
We identified the slow query in IRC today and Gabriel is working on a fix, last I checked.
Comment 10 Thomas Pifer 2008-08-02 04:29:57 UTC
(In reply to comment #7)
> (In reply to comment #5)
> > 1. Tools->Cover Art->Download Cover Art
> > 2. Try to change to next track while fetching occurs
> > 
> > Hang.
> 
> Can anybody else reproduce this with r4254 or later?
> 

Can reproduce using 1.2 on Hardy.
Comment 11 Andrew Conkling 2009-03-10 12:43:28 UTC
Hearing reports about this downstream: https://bugs.launchpad.net/bugs/340436.

I've noticed that "Downloading Cover Art" takes a long time when it is triggered by rescanning my library, even on recent SVN checkouts. Would --debug-sql output be useful here or is there otherwise any information others or I could provide to track this down?
Comment 12 André Klapper 2020-03-17 08:24:40 UTC
Banshee is not under active development anymore and had its last code changes more than three years ago. Its codebase has been archived.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this ticket (or rather transfer the project
to GNOME Gitlab, as GNOME Bugzilla is being shut down) if anyone takes the
responsibility for active development again.
See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/264 for more info.