GNOME Bugzilla – Bug 749790
Make Metadata harvester a process
Last modified: 2015-07-04 11:01:44 UTC
That way, if a broken file crashes GStreamer or libmediaart, we can - gracefully revover - blacklist such a file - give precise information about a problematic file instead of poking in the dark with debug info as we do currently Issues: - That's duplicating Tracker/LMS architecture, but I'm sick of the bugs - How to get meta-data to calling process? - gst_discoverer_info_to_variant - quite recent, not sure what is serialized - Custom text-based format with the necessary meta-data - process would do the DLNA guessing and media-art extraction as well - direct write into SQLite db - potentially problematic, as we might have many writers in upload use-case
Created attachment 305873 [details] [review] media-export: Extract meta-data in external process Move GStreamer calls to extract meta-data into own process. This enables us to recover from crashes due to bugs with broken media files, knowing exactly which URI caused the crash and (later) blacklist the broken files.
Comment on attachment 305873 [details] [review] media-export: Extract meta-data in external process First step. Has some regressions such as - MetaData extraction cannot be disabled - Blacklisting of failing uris is missing - AlbumArt extraction is missing - There is one process per harvesting task. These will be addresses in the next release. Features are: - Process is restarted if it is killed - URI that made the extraction process fail is known and shown to the user - Extraction process is run at nice level 19 which should lower system load during initial crawl. Attachment 305873 [details] pushed as 1e1dc09 - media-export: Extract meta-data in external process
Created attachment 306797 [details] [review] media-export: Re-add album art extraction Signed-off-by: Jens Georg <mail@jensge.org>
Created attachment 306798 [details] [review] media-export: Listen to meta-data config again Signed-off-by: Jens Georg <mail@jensge.org>
More missing features