Bug #4066
closedMulticast strategy retransmission suppression does not take face into account
100%
Description
An application (ex: sync protocol) may send an interest I1 at time t1 and receive an interest I2 at time t2 from the same face within a short period of time (t2 - t1 < 10 ms).
By current implementation of retransmission suppression with respect to pitEntry's latest outRecord time, I2 will be suppressed.
This issue will implement a per upstream retransmission suppression as suggested in note1 and use it in multicast strategy (Not sure if other strategies need it).
Files
Updated by Junxiao Shi over 7 years ago
In multicast logic we should suppress only if all the faces already have an interest sent out.
This is insufficient. In topology below, if both B and C express an Interest within a short period, A would receive the Interest twice, which is unnecessary overhead.
A
|
B-R-C
The suppression timer should be enforced per upstream rather than per PIT entry. The Interest should be forwarded to an upstream peer if no Interest has been forwarded to it within last "suppression period".
Updated by Ashlesh Gawande over 7 years ago
- Status changed from New to In Progress
Updated by Junxiao Shi over 7 years ago
- Assignee set to Ashlesh Gawande
- Target version set to v0.6
Updated by Ashlesh Gawande over 7 years ago
- File per_pit_entry_suppression_problem.jpg per_pit_entry_suppression_problem.jpg added
- File deleted (
psync.jpg) - Description updated (diff)
Updated description for clarity.
Updated by Ashlesh Gawande over 7 years ago
- % Done changed from 0 to 90
Will try to update NFD dev guide soon.
Also, is this functionality desired in other strategies like best route strategy or per pit entry suppression is good enough?
Updated by Ashlesh Gawande over 7 years ago
- % Done changed from 90 to 100
Have updated the NFD dev guide, can someone please review:
https://gitlab.com/named-data/TR-NDN-0021-NFD-dev-guide/commit/bd5ab925164dcab550654459e01fcd94cfcb56e4
Updated by Ashlesh Gawande almost 7 years ago
- Status changed from In Progress to Closed
Updated by Ashlesh Gawande almost 4 years ago
- Related to Feature #5140: Add per upstream suppression to ASF strategy added