BIND 10 #347: Implement query counters in auth module
BIND 10 Development
do-not-reply at isc.org
Thu Dec 2 12:01:29 UTC 2010
#347: Implement query counters in auth module
------------------------------+---------------------------------------------
Reporter: naokikambe | Owner: jinmei
Type: enhancement | Status: reviewing
Priority: major | Milestone: y2 12 month milestone
Component: b10-auth | Resolution:
Keywords: | Sensitive: 0
Estimatedhours: 0.0 | Hours: 0
Billable: 1 | Totalhours: 0
Internal: 0 |
------------------------------+---------------------------------------------
Changes (by y-aharen):
* owner: naokikambe => jinmei
Comment:
Jinmei-san, thank you very much for your review. Before fixing the code
and committing to branch, I have few points to be clear.
Replying to [comment:5 jinmei]:
> '''stats.cc'''
> - The comment and log messages assume that the other end is the
> b10-stats program, but interface-wise it's not necessarily correct
> (that could be a third-party app that implements the stat API,
> right?)
Counter in Auth module is not intended to communicate with third-party
stats module. It only communicates with b10-stats.
> - BTW, even if it doesn't cause timeout and exception, we should
> actually not even block in group_recvmsg (and also
> group_sendmsg(), which can happen because it's a connected
> channel): b10-auth shouldn't stop handling DNS queries even for 1
> second. Since it should relatively a rare event we can live with
> the current blocking mode (and there are actually other parts of
> b10-auth that have the same problem). But please leave a TODO
> comment on this point. We should eventually solve it.
I don't think that blocking in group_{send,recv}msg also blocks handling
of DNS queries. I understand blocking read/write operation stops in the
ASIO library, and if another event arrives ASIO starts handling it. Excuse
me if I misunderstood.
--
Ticket URL: <http://bind10.isc.org/ticket/347#comment:8>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list