Opened 5 years ago
#3300 new bug
Move Download Folder triggers recheck even if the new folder is the same as the old one
Reported by: | hfuss | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | needs verified |
Component: | Core | Version: | 2.0.3 |
Keywords: | Cc: |
Description
I'm not sure if the recheck on move download folder is intentional, but I can kind of understand why it may be desirable since move operations typically complete when they're committed to filesystem cache, not disk. If that's the intention though, writes may still not be committed to disk for the reads related to the recheck.
I haven't actually looked at the deluge code for this, but am assuming it's using os.rename?
- If the location is the same, do nothing (deluge 1.3 behavior)
- If the files aren't actually moving (same physical device), don't run recheck. This is just a couple of inode updates, so should be pretty safe. (deluge 1.3 behavior)
- When moving to a new location, unless there's a Python function that already does this, you may want to do the move yourself so that you can fsync. In that case, the recheck should not be necessary. Opening the fd after the move and running fsync may also work, but I'm not sure (it may only try to flush the writes you've done since opening the descriptor rather than all dirty blocks for the file)
Note:
See TracTickets
for help on using tickets.