Task #4887
closed
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.
ndn-tools and ndn-cxx implementations are exactly the same except that for ndn-tools has an afterMeasurement signal.
- 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
- Status changed from In Progress to Code review
- Target version set to v0.7
- % Done changed from 0 to 60
- % Done changed from 60 to 80
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.
- Tracker changed from Feature to Task
- Status changed from Code review to Closed
- % Done changed from 80 to 100
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by