BIND 10 #1958: Implement perfdhcp statistics component
BIND 10 Development
do-not-reply at isc.org
Tue Aug 7 14:38:28 UTC 2012
#1958: Implement perfdhcp statistics component
-------------------------------------+-------------------------------------
Reporter: | Owner: stephen
stephen | Status: reviewing
Type: | Milestone: Sprint-
enhancement | DHCP-20120730
Priority: | Resolution:
medium | Sensitive: 0
Component: | Sub-Project: DHCP
perfdhcp | Estimated Difficulty: 16
Keywords: | Total Hours: 67
Defect Severity: N/A |
Feature Depending on Ticket: |
Add Hours to Ticket: 2 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Changes (by marcin):
* hours: 9 => 2
* owner: marcin => stephen
* totalhours: 65 => 67
Comment:
I corrected the code as suggested.
Regarding the rcvd_packets_num_ counter....
The fact that increment operation was moved from appendRcvd() to
findSent() (which has been now renamed to matchPackets()) does not change
the behavior. This is because appendSent() was called only in case the
received packet was matched with sent packet. I had to move this out from
appendRcvd() because when so called 'archive mode' is disabled I neither
store sent nor received packets once they are matched. In other words I
never call appendRecvd() when archive mode is disabled but I still want to
increment the counter. It seems to be the most natural place to do it
where it is done now.
I also decided to rename the findSent() function to matchPackets() which
in my opinion better reflects the purpose of it. Not only does it find the
'sent' packet but also it does other things like increment counters.
--
Ticket URL: <http://bind10.isc.org/ticket/1958#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list