Task #4887
closed
ASF Strategy should estimate RTO using TCP’s algorithm (RFC6298) instead of NFD’s RtoEstimator
100%
Description
Reference implementation from catchunks:
https://github.com/named-data/ndn-tools/blob/master/tools/chunks/catchunks/rtt-estimator.cpp
Updated by Davide Pesavento over 5 years ago
We have 3 different implementations of RttEstimator in 3 different places (ndn-cxx, ndn-tools, NFD). We should consolidate them.
For this task, I suggest fixing/improving the ndn-cxx implementation and then using that one in both NFD and ndncatchunks.
Updated by Ernest McCracken over 5 years ago
ndn-tools and ndn-cxx implementations are exactly the same except that for ndn-tools has an afterMeasurement signal.
Updated by Ernest McCracken over 5 years ago
- Status changed from New to In Progress
Yes. I will open a gerrit code review soon. We fixed it by having Asf strat use ndn-cxx's rto estimator. We don't need any fixes to ndn-cxx's rto estimator but I will have to check to see what all touches NFD's rto estimator and what changes might be needed there
Updated by Davide Pesavento over 5 years ago
- Status changed from In Progress to Code review
- Target version set to v0.7
Updated by Junxiao Shi over 5 years ago
For AccessStrategy, it needed a state-copy operator, so that per-prefix RTT estimator can inherit from per-face RTT estimator state. The rationale is explained in NFD devguide.
If this operator cannot be supported, it's okay to drop it, but strategy version number must be incremented as this represents a behavior change.
Updated by Davide Pesavento over 5 years ago
- Tracker changed from Feature to Task
- Status changed from Code review to Closed
- % Done changed from 80 to 100