Bug #2621
closedInterest aggregation scenario: broken due to retransmission suppression change
100%
Description
In IntegrationTests, Interest aggregation scenario (#1384) assumes NFD suppresses all similar Interests when a PIT entry is pending.
When strategies start to permit consumer retransmission, similar Interests are forwarded if they are not too frequent.
With exponential backoff suppression algorithm, the default setting would allow retransmission after only 1ms.
Interest aggregation scenario is broken due to this change, and needs to be redesigned.
Files
Updated by Junxiao Shi over 9 years ago
- Assignee set to Junxiao Shi
I'll scratch the design. After that, this issue will be reassigned for implementation.
Updated by Junxiao Shi over 9 years ago
- Related to Task #1384: Interest aggregation test scenario added
Updated by Junxiao Shi over 9 years ago
- Related to Feature #1913: RetransmissionSuppression: exponential back-off added
Updated by Junxiao Shi over 9 years ago
- Description updated (diff)
- Assignee deleted (
Junxiao Shi)
I suggest this scenario to be fixed by explicitly setting best-route v1 strategy (ndn:/localhost/nfd/strategy/best-route/%FD%01
) which does not allow consumer retransmission.
Other possible solutions are:
- Allow passing parameters to strategies, so that the initial interval of consumer retransmission can be controlled from the scenario. This is significantly more complex than using best-route v1, and duplicates some work of #2000.
- Disable this scenario. This is clearly not the best solution.
Updated by Junxiao Shi over 9 years ago
- Assignee set to Eric Newberry
20150727 conference call approves the design in note-4: set strategy to ndn:/localhost/nfd/strategy/best-route/%FD%01
.
Updated by Eric Newberry over 9 years ago
- Status changed from New to In Progress
Updated by Eric Newberry over 9 years ago
- Status changed from In Progress to Code review
Updated by Junxiao Shi over 9 years ago
- File 20150729194504.tgz 20150729194504.tgz added
Updated by Junxiao Shi over 9 years ago
- Blocked by Bug #3073: integ broken due to ndn-cxx shared library added
Updated by Junxiao Shi over 9 years ago
- File 20150729234249.tgz 20150729234249.tgz added
- Status changed from Code review to Closed
- % Done changed from 0 to 100