View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009284 | MMW v4 | Synchronization | public | 2012-04-20 21:22 | 2012-06-04 23:31 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | major | Reproducibility | sometimes |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.0.5 | ||||
| Target Version | 4.1 | Fixed in Version | 4.0.5 | ||
| Summary | 0009284: MTP performance can be very slow in some cases | ||||
| Description | There are numerous reports that MTP synchronization is unusably slow--especially for certain android devices. I have not been able to replicate this, but it seems to be specific to certain android devices in mtp mode. | ||||
| Additional Information | http://www.mediamonkey.com/forum/viewtopic.php?f=6&t=64066&start=15 | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1493 | ||||
| related to | 0006491 | closed | Ludek | MTP Devices should use root folder for in-place configuration |
| related to | 0002615 | closed | Ludek | MTP Synchronization is MUCH slower than it should be |
| related to | 0007580 | closed | Ludek | MTP Synchronization takes ~5-10s to initiate (regression) |
| related to | 0002009 | resolved | MTP Device Export: performance is very slow | |
| related to | 0009368 | closed | Ludek | Improved UI for bi-di sync |
|
|
Per e-mail discussion, for 4.0.5 we can modify the scanning to be a little more 'smart'. Particularly for Android devices (recognized by root Android folder, or directly by device profiles in WMDM) we should avoid scanning Android, DCIM, Backups and possibly some other folders that are used by some popular applications and can contain a lot of files/sub-folders. |
|
|
I was also thinking about one addition to the "smart scan": If MM is traversing folder that has say 2000 images (like DCIM) then MM could be smart enough and if it hasn't recognize a known file (audio or video) between first 20 files then it could simply skip the folder. This way we could speed things a lot. |
|
|
Added ignoring of several common android folders in build 1490. 'Android', 'zemail', '.android_secure', 'Aura', 'DCIM', 'LOST.DIR', 'Pictures', 'soundhound' Waiting for logs to add others... |
|
|
Note that DCIM shouldn't be excluded, because it can contains MP4 videos that wouldn't be visible by MM then. |
|
|
Fixed in build 1490. |
|
|
Added some other ignore folders (based on logs) like 'Wallpapers' into the build 1491. |
|
|
Watching the logs it makes me think that we should probably really limit MTP scanning to only sync targets even for 4.0.5, because in one log it takes __several__minutes__ to scan folder '.mobilenoter', it is by evidence a folder created by an app called MobileNoter, but we can hardly include all such a apps to the ignored folders set. So I would suggest to limit the scanning to sync targets only at least for all android devices connected in MTP mode, what do you think? |
|
|
It makes sense, so long as we provide a mechanism for adding other folders to the scan list. We can re-use the 'Select folders you want to scan for media files' string if needed... |
|
|
Just to summarize: The reasons to prefer scanning of whole device were that 0006491 shows device content based on the device scan. If we would scan only sync targets then the amount of Video/Audio/Other at the bottom graphical indicator would be inaccurate, only the songs in the sync targets would be shown under the Device->Music node, etc. If the user would change the sync targets then the content presented in the old targets would not be shown at all and this would be definetly an issue. There would be also issues with the background sync then. So limiting device scanning only to currently configured sync targets is buggy. On the other hand as it is definetly horribly slow for some users we should give them an option to scan only sync targets. This will be definetly fixed for 4.1 (as already discussed) to be in beta testing soon. We cannot change this for 4.0.5 because of two reasons: 1) We won't have enough time to properly assess the downside (i.e. we don't know and won't know by then how many MTP users rely on scanning directories that aren't configured as target folders). 2) Strings translation is frozen, we cannot easily add new UI for this and it also doesn't make much sense as 4.1 will be in beta testing soon. |
|
|
Added several others folders to ignore in build 4.0.5.1492 (based on logs). |
|
|
Significantly sped up metadata scanning for tracks synced by MM in build 1493. |
|
|
Verified 1495 |
