Feedback on svn-1348

Viewing 15 posts - 1 through 15 (of 19 total)
  • Author
    Posts
  • #509
    grommet
    Participant

    Brilliant speed, Ron! svn-1344 (win32) @ debug 9 logging with iTunes XML active:

    2006-08-14 21:15:07 (f3f11f6d): Initializing database
    2006-08-14 21:15:07 (f3f11f6d): Full reload...
    2006-08-14 21:15:07 (f3f11f6d): Starting mp3 scan
    2006-08-14 21:17:25 (f3f11f6d): Ending song scan
    2006-08-14 21:17:27 (f3f11f6d): Processing playlists
    2006-08-14 21:17:27 (f3f11f6d): Starting playlist scan
    2006-08-14 21:17:54 (f3f11f6d): Finished playlist loop
    2006-08-14 21:17:59 (f3f11f6d): Serving 20679 songs. Startup complete in 172 seconds

    So, speed problem fixed. 😀 More news as it develops…

    #5936
    grommet
    Participant

    One more benchmark with iTunes XML, now without debug logging:

    2006-08-14 22:13:05 (f3f11f6d): Full reload...
    2006-08-14 22:13:05 (f3f11f6d): Starting mp3 scan
    2006-08-14 22:14:10 (f3f11f6d): Starting playlist scan
    2006-08-14 22:14:10 (f3f11f6d): Done processing playlist
    2006-08-14 22:14:37 (f3f11f6d): Scanned 20679 songs in 92 seconds
    2006-08-14 22:14:37 (f3f11f6d): Serving 20679 songs. Startup complete in 92 seconds

    Just a tad over 90 seconds for a ~20K track full load from scratch. WOW!

    I thought I had some iTunes client Playlist sorting issues, but they have since disappeared. So, maybe it was my iTunes client losing it’s mind. (Side note: I still get the Playlist entry bad: Path not found errors in the log on the blank lines in the .M3U files…)

    #5937
    grommet
    Participant

    With debug=0, “Rescanning database” messaages are logged… but still no finish/”scanned” message. IMHO, if you log the scan start with debug=0… you should log the end. (Also see “Starting playlist scan” messages without completion, but that’s OK since it’s part of the rescan.)

    #5938
    rpedde
    Participant

    @grommet wrote:

    With debug=0, “Rescanning database” messaages are logged… but still no finish/”scanned” message. IMHO, if you log the scan start with debug=0… you should log the end. (Also see “Starting playlist scan” messages without completion, but that’s OK since it’s part of the rescan.)

    I thought I did. 🙂 Dammit!

    — Ron

    #5939
    grommet
    Participant

    Ok, the scan completion message is fixed in svn-1348. 😀 But I do still get those silly path errors in the log from my perfectly fine .M3U files. 8)

    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found
    2006-08-16 23:13:10 (f3f11f6d): Playlist entry bad: Path not found

    #5940
    sonichouse
    Participant

    Minor issue for you ….
    The web configuration page does not handle the new case_sensitive flag.

    The options below are in your mt-daapd.conf and Firefly uses them, but this web page can’t handle them until they are added to config.xml
    general:case_sensitive

    Steve

    #5941
    grommet
    Participant

    In my svn-1348 (win32), if I try to toggle Case Sensitive via config.html and click save, I get the following error:

    Error: 500scanning:case_sensitive

    It also said it was “Yes” by default on Win32… Is this correct?

    And while I’m on config.html, I see “Should playlists be returned in the order specified in ________ the playlist?” — there is a long blank space before the playlist? Your config.xml has a carriage return there.

    229             
    230> Should playlists be returned in the order specified in
    231> the playlist?
    232>
    #5942
    rpedde
    Participant

    @sonichouse wrote:

    Minor issue for you ….
    The web configuration page does not handle the new case_sensitive flag.

    The options below are in your mt-daapd.conf and Firefly uses them, but this web page can’t handle them until they are added to config.xml
    general:case_sensitive

    Steve

    That’s because it belongs in [scanning], not [general]. 🙂

    — Ron

    #5943
    rpedde
    Participant

    @grommet wrote:

    In my svn-1348 (win32), if I try to toggle Case Sensitive via config.html and click save, I get the following error:

    Error: 500scanning:case_sensitive

    It also said it was “Yes” by default on Win32… Is this correct?

    And while I’m on config.html, I see “Should playlists be returned in the order specified in ________ the playlist?” — there is a long blank space before the playlist? Your config.xml has a carriage return there.

    229             
    230> Should playlists be returned in the order specified in
    231> the playlist?
    232>

    Argh… emacs ate my xml file.

    #5944
    grommet
    Participant

    Ack! My first punt in many builds. Regretably, I was not at a higher debug level… but maybe that wouldn’t have helped, either. Hopefully a fluke?

    2006-08-18 21:46:19 (58149bda): cannot unlock sqlite2 lock: Incorrect function.

    2006-08-18 21:46:19: Aborting

    Also, earlier in the log… I had a few non-fatal errors logged. Not sure what they mean:

    2006-08-17 00:21:09 (2fd9b67f): Thread 633: Entering ws_returnerror (302: Moved)
    2006-08-17 00:30:41 (6cb17b20): Thread 742: Entering ws_returnerror (302: Moved)
    2006-08-17 02:52:31 (2fa1d05b): Thread 2378: could not read: The operation completed successfully.
    2006-08-17 03:30:27 (1c47f7ab): Thread 2674: Entering ws_returnerror (302: Moved)
    2006-08-17 22:59:16 (6ca7186f): Thread 2757: could not read: The operation completed successfully.
    2006-08-18 10:57:55 (1c57dea8): Thread 2785: could not read: The operation completed successfully.
    2006-08-18 21:41:04 (6ca7a4d0): Thread 2773: could not read: The operation completed successfully.
    #5945
    fizze
    Participant

    well, this is a little OT…

    My playlists, that worked with 1200something dont work with 1348.
    I set the case sensitive = 1, and its recognized by the web interface, so I guess its right.

    Just triggered a full rescan from the interface, maybe that helps.

    Other than that, any clues ?

    #5946
    grommet
    Participant

    Set debug logging to 9 and see what happens when it walks the Playlist? I can’t repro this. But I don’t have a lot of playlists on my test environment.

    #5947
    rpedde
    Participant

    @fizze wrote:

    well, this is a little OT…

    My playlists, that worked with 1200something dont work with 1348.
    I set the case sensitive = 1, and its recognized by the web interface, so I guess its right.

    Just triggered a full rescan from the interface, maybe that helps.

    Other than that, any clues ?

    what kind of playlists? m3u, itunes, or smart?

    #5948
    grommet
    Participant

    I think I repro’d part of the broken playlist problem using an old build and upgrading to the latest nightly… keeping the old database. I guess I’ve been blowing away databases too much. 8)

    #5949
    rpedde
    Participant

    @grommet wrote:

    I think I repro’d part of the broken playlist problem using an old build and upgrading to the latest nightly… keeping the old database. I guess I’ve been blowing away databases too much. 8)

    Yeah, me too. Found it, though. Full scans through the web interface weren’t forcing scans on playlists, and I should have forced a full scan anyway when I added the case stuff. In any event, it’s fixed in svn now.

    Stopping the server, whacking the db, and starting the server will also fix it, as a short-term fix.

    — Ron

Viewing 15 posts - 1 through 15 (of 19 total)
  • The forum ‘Nightlies Feedback’ is closed to new topics and replies.