More and better error messages in the DHT.
[quix0rs-apt-p2p.git] / TODO
1 Reduce the memory footprint by clearing the AptPackages caches.
2
3 The memory usage is a little bit high due to keeping the AptPackages
4 caches always. Instead, they should timeout after a period of inactivity
5 (say 15 minutes), and unload themselves from meory. It only takes a few
6 seconds to reload, so this should not be an issue.
7
8
9 Packages.diff files need to be considered.
10
11 The Packages.diff/Index files contain hashes of Packages.diff/rred.gz 
12 files, which themselves contain diffs to the Packages files previously 
13 downloaded. Apt will request these files for the testing/unstable 
14 distributions. They need to either be ignored, or dealt with properly by 
15 adding them to the tracking done by the AptPackages module.
16
17
18 PeerManager needs to download large files from multiple peers.
19
20 The PeerManager currently chooses a peer at random from the list of 
21 possible peers, and downloads the entire file from there. This needs to 
22 change if both a) the file is large (more than 512 KB), and b) there are
23 multiple peers with the file. The PeerManager should then break up the 
24 large file into multiple pieces of size < 512 KB, and then send requests 
25 to multiple peers for these pieces.
26
27 This can cause a problem with hash checking the returned data, as hashes 
28 for the pieces are not known. Any file that fails a hash check should be 
29 downloaded again, with each piece being downloaded from different peers 
30 than it was previously. The peers are shifted by 1, so that if a peers 
31 previously downloaded piece i, it now downloads piece i+1, and the first 
32 piece is downloaded by the previous downloader of the last piece, or 
33 preferably a previously unused peer. As each piece is downloaded the 
34 running hash of the file should be checked to determine the place at 
35 which the file differs from the previous download.
36
37 If the hash check then passes, then the peer who originally provided the 
38 bad piece can be assessed blame for the error. Otherwise, the peer who 
39 originally provided the piece is probably at fault, since he is now 
40 providing a later piece. This doesn't work if the differing piece is the 
41 first piece, in which case it is downloaded from a 3rd peer, with 
42 consensus revealing the misbehaving peer.
43
44
45 Store and share torrent-like strings for large files.
46
47 In addition to storing the file download location (which would still be 
48 used for small files), a bencoded dictionary containing the peer's 
49 hashes of the individual pieces could be stored for the larger files 
50 (20% of all the files are larger than 512 KB). This dictionary would 
51 have the normal piece size, the hash length, and a string containing the 
52 piece hashes of length <hash length>*<#pieces>. These piece hashes could 
53 be compared ahead of time to determine which peers have the same piece 
54 hashes (they all should), and then used during the download to verify 
55 the downloaded pieces.
56
57 For very large files (5 or more pieces), the torrent strings are too 
58 long to store in the DHT and retrieve (a single UDP packet should be 
59 less than 1472 bytes to avoid fragmentation). Instead, the peers should 
60 store the torrent-like string for large files separately, and only 
61 contain a reference to it in their stored value for the hash of the 
62 file. The reference would be a hash of the bencoded dictionary. If the 
63 torrent-like string is short enough to store in the DHT (i.e. less than 
64 1472 bytes, or about 70 pieces for the SHA1 hash), then a 
65 lookup of that hash in the DHT would give the torrent-like string. 
66 Otherwise, a request to the peer for the hash (just like files are 
67 downloaded), should return the bencoded torrent-like string.
68
69
70 PeerManager needs to track peers' properties.
71
72 The PeerManager needs to keep track of the observed properties of seen 
73 peers, to help determine a selection criteria for choosing peers to 
74 download from. Each property will give a value from 0 to 1. The relevant 
75 properties are:
76
77  - hash errors in last day (1 = 0, 0 = 3+)
78  - recent download speed (1 = fastest, 0 = 0)
79  - lag time from request to download (1 = 0, 0 = 15s+)
80  - number of pending requests (1 = 0, 0 = max (10))
81  - whether a connection is open (1 = yes, 0.9 = no)
82
83 These should be combined (multiplied) to provide a sort order for peers 
84 available to download from, which can then be used to assign new 
85 downloads to peers. Pieces should be downloaded from the best peers 
86 first (i.e. piece 0 from the absolute best peer).
87
88
89 When looking up values, DHT should return nodes and values.
90
91 When a key has multiple values in the DHT, returning a stored value may not
92 be sufficient, as then no more nodes can be contacted to get more stored
93 values. Instead, return both the stored values and the list of closest
94 nodes so that the peer doing the lookup can decide when to stop looking
95 (when it has received enough values).
96
97 Instead of returning both, a new method could be added, "lookup_value".
98 This method will be like "get_value", except that every node will always
99 return a list of nodes, as well as the number of values it has for that
100 key. Once a querying node has found enough values (or all of them), then
101 it would send the "get_value" method to the nodes that have the most
102 values. The "get_value" query could also have a new parameter "number",
103 which is the maximum number of values to return.
104
105
106 Missing Kademlia implementation details are needed.
107
108 The current implementation is missing some important features, mostly 
109 focussed on storing values:
110  - values need to be republished (every hour?)