Feature #4299
closedndncatchunks: Print summary for all pipeline types
Added by Anonymous about 7 years ago. Updated almost 7 years ago.
100%
Description
Currently, the summary only works for the AIMD pipeline. The code should be refactored to also make it work for the fixed-window or any other pipeline.
Updated by Anonymous about 7 years ago
This is just a simple task for Chavoosh to get started with the coding procedure.
Updated by Davide Pesavento about 7 years ago
- Tracker changed from Bug to Feature
- Subject changed from ndncatchunks: Print summary for all pipelines to ndncatchunks: Print summary for all pipeline types
- Description updated (diff)
The function PipelineInterestsAimd::printSummary() should be moved to PipelineInterests::printSummary()
It's not so simple. Several quantities printed by PipelineInterestsAimd::printSummary
are not available in the base class. Moreover, the fixed-window pipeline simply does not track things such as the # of loss events and the # of retransmitted Interests, so it cannot print them.
Shuo started doing this, not sure in what state the code was but you may take a look: https://gerrit.named-data.net/3746
Updated by Anonymous about 7 years ago
Davide Pesavento wrote:
The function PipelineInterestsAimd::printSummary() should be moved to PipelineInterests::printSummary()
It's not so simple. Several quantities printed by
PipelineInterestsAimd::printSummary
are not available in the base class. Moreover, the fixed-window pipeline simply does not track things such as the # of loss events and the # of retransmitted Interests, so it cannot print them.
Totally agree that it's not quite as simple.
For getting Chavoosh started, I think it's fine to just have some basic output for all pipelines (like completion time and throughput). And then the specific pipeline (like AIMD) can overwrite the printSummary() to include parts that are not available in other pipelines.
Shuo started doing this, not sure in what state the code was but you may take a look: https://gerrit.named-data.net/3746
Good point.
Updated by Chavoosh Ghasemi about 7 years ago
- Status changed from New to Code review
- % Done changed from 0 to 100
Updated by Chavoosh Ghasemi about 7 years ago
Klaus Schneider wrote:
Davide Pesavento wrote:
The function PipelineInterestsAimd::printSummary() should be moved to PipelineInterests::printSummary()
It's not so simple. Several quantities printed by
PipelineInterestsAimd::printSummary
are not available in the base class. Moreover, the fixed-window pipeline simply does not track things such as the # of loss events and the # of retransmitted Interests, so it cannot print them.Totally agree that it's not quite as simple.
For getting Chavoosh started, I think it's fine to just have some basic output for all pipelines (like completion time and throughput). And then the specific pipeline (like AIMD) can overwrite the printSummary() to include parts that are not available in other pipelines.
Shuo started doing this, not sure in what state the code was but you may take a look: https://gerrit.named-data.net/3746
Good point.
Davide Pesavento wrote:
The function PipelineInterestsAimd::printSummary() should be moved to PipelineInterests::printSummary()
It's not so simple. Several quantities printed by
PipelineInterestsAimd::printSummary
are not available in the base class. Moreover, the fixed-window pipeline simply does not track things such as the # of loss events and the # of retransmitted Interests, so it cannot print them.Shuo started doing this, not sure in what state the code was but you may take a look: https://gerrit.named-data.net/3746
Tracking statistical info of packet delivery is still is pipelines' responsibility. E.g. if AIMD wants to use printSummary() method it should gather statics of packet delivery and pass them to printSummary() method. Simply put, this method is nothing more than a printer, where receives its info through arguments.
I have moved printSummary method to pipeline-interests.cpp file. However, I cannot push my changes. I will fix the problem and push them.
Updated by Anonymous about 7 years ago
Were you able to push to gerrit?
You can post your error message here, so we can help.
Updated by Anonymous about 7 years ago
- Related to Feature #4289: ndncatchunks: React to congestion marks added
Updated by Anonymous about 7 years ago
In code review: https://gerrit.named-data.net/c/4220/
Updated by Davide Pesavento about 7 years ago
- Related to deleted (Feature #4289: ndncatchunks: React to congestion marks)
Updated by Davide Pesavento about 7 years ago
- Blocks Feature #4289: ndncatchunks: React to congestion marks added
Updated by Davide Pesavento almost 7 years ago
- Status changed from Code review to Closed
- % Done changed from 50 to 100