Media is reindexed each time backend is started

Bug #236474 reported by Joshua Scotton
12
Affects Status Importance Assigned to Milestone
Entertainer Media Center
Fix Released
High
Paul Hummer

Bug Description

Media is reindexed each time the backend is started.

The desired behaviour should be the backend just checking for changes to the existing index of movies, music etc.

Tags: new-backend
Paul Hummer (rockstar)
Changed in entertainer:
assignee: nobody → rockstar
importance: Undecided → High
status: New → Triaged
Paul Hummer (rockstar)
Changed in entertainer:
milestone: none → entertainer-0.2
Paul Hummer (rockstar)
Changed in entertainer:
status: Triaged → In Progress
Paul Hummer (rockstar)
Changed in entertainer:
milestone: entertainer-0.2 → none
Revision history for this message
Paul Hummer (rockstar) wrote :

I think it's important that I document on this bug where we are with this process. The ImageCache work is completed on the backend, and just needs a bit more forward porting on the frontend. However, I now have more than a deep understanding of why exactly it re-indexes everything. There are a few reasons for this:

1) When calling update on the old data model, it deleted everything and reindexed everything.
2) On each startup, MediaCacheManager.rebuild*Cache is called which deletes and re-creates this.

The fix is actually a lot more trivial than I was expecting it to be. I plan on fixing this after all the cache classes have been ported to the new data model.

As a side note, tracking this down was nice, because it clearly laid out an upgrade path for new iterations of the data model. We can harness this same concept to trigger when a table needs to be rebuilt with a new structure. Blessing in disguise (and, I'm starting to think, was the original idea behind this constant re-indexing).

Changed in entertainer:
milestone: none → entertainer-0.3
Paul Hummer (rockstar)
Changed in entertainer:
milestone: entertainer-0.3 → none
status: In Progress → Triaged
milestone: none → entertainer-0.3
status: Triaged → In Progress
Revision history for this message
Matt Layman (mblayman) wrote :

I'm not marking this as Fix Released because the code is only on the 0.3 branch and not the trunk. I think we can release it after we merge 0.3 fixes onto trunk.

Changed in entertainer:
status: In Progress → Fix Committed
Paul Hummer (rockstar)
Changed in entertainer:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.