bugmldonkey, a multi-networks file-sharing client - Bugs: bug #23518, Verify chunks causes bad...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #23518: Verify chunks causes bad statistics on file

Submitter:  Renato Alves <rjalves>
Submitted:  Sat 07 Jun 2008 04:04:34 PM UTC
   
 
Category:  Core Severity:  3 - Normal
Item Group:  Program malfunction Status:  None
Assigned to:  None Open/Closed:  Open
Release:  None Release:  2.9.5
Operating System:  Linux Binaries Origin:  Gentoo ebuild
CPU type:  Intel x86

Sun 25 Apr 2010 08:58:40 PM UTC, comment #3: 

I think patch #6401 is wrong anyway and should be reverted (and better solution found for that problem), it counts the data as downloaded when it is not the case - i.e. it already resides on disk, maybe it was a partial download from another client. The only way to count the data as downloaded is when it is read from the socket or else such strange results will occur. Counting of downloaded data shouldn't be done in swarmer - do you agree? Rather we should have two counters - downloaded and available bytes. First is counted somewhere near network reading, second is tracked by swarmer.

For the bug numbers, agreed, though I personally would have done it another way round (bugs as primary entities and patches linked to bugs (not bugs promoted to patches)).

ygrek <ygrek>
Group Member
Sat 10 Apr 2010 04:40:12 PM UTC, comment #2: 

Basically you are reverting patch #6401 here which incompletely fixed another bug.

Regarding the bug numbers, I think it is not really necessary because you can see the former bug number of a patch in the patch history, like here:
http://savannah.nongnu.org/patch/index.php?6961#history

spiralvoice <spiralvoice>
Group administrator
Sat 10 Apr 2010 01:35:10 PM UTC, comment #1: 

Fix attached.
BTW I'd suggest adding to changelog entries bug numbers that were closed by the patch.

(file #20181)

ygrek <ygrek>
Group Member
Sat 07 Jun 2008 04:04:34 PM UTC, original submission:  

Added a ed2k file that downloaded for a couple of days.

Using the web interface:

After some download time and a couple of sessions I clicked the filename and then verify chunks to make sure everything was going ok.

Doing this caused the file statistics to be displayed incorrectly.
One of the files displayed information like the following:

size    DLed    %        ETA
26.1M 35.4M 135.6 -1:-40

This didn't affect the file itself since eventually the download ended without problems nor corruption.

Meanwhile I tried the same with other files and in all of them the same happened.

This problem only affects the current session. Restarting mldonkey causes the files to display the correct info again.

Renato Alves <rjalves>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #20181:  verify_chunks.diff added by ygrek (575B - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by spiralvoice (Posted a comment)
  • -email is unavailable- added by ygrek (Updated the item)
  • -email is unavailable- added by rjalves (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-04-10 ygrek Attached File- Added verify_chunks.diff, #20181

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code