BIND 10 #2212: support "load zone" command in the configurator thread
BIND 10 Development
do-not-reply at isc.org
Fri Oct 26 00:00:28 UTC 2012
#2212: support "load zone" command in the configurator thread
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: accepted
Type: task | Milestone:
Priority: | Sprint-20121106
medium | Resolution:
Component: | Sensitive: 0
b10-auth | Sub-Project: DNS
Keywords: | Estimated Difficulty: 5
Defect Severity: N/A | Total Hours: 0
Feature Depending on Ticket: |
background zone loading |
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
This task is ready for review at the trac2212-2 branch (note: not
"trac2212", which has also been pushed).
I first merged a snapshot of trac2209 (03499b1), which should be
ignored.
The main changes (in datasrc_clients_mgr.h) should be pretty
straightforward. For tests, I mainly ported current tests using
the old interface in command_unittest. Porting them is mostly
straightforward, but see commit logs starting with "ported command
test".
There's one thing I may have to update when trac2209 is merged:
{{{#!cpp
case datasrc::ConfigurableClientList::ZONE_RELOADED: // XXX misleading
name
assert(writerpair.second);
return (writerpair.second);
}}}
IMO this result code is misleading, and I pointed it out in my review
for trac2209. If we agree on changing this, we'll need to update this
code too. But this should be a trivial adjustment and I believe can
be done on merge of this branch.
--
Ticket URL: <http://bind10.isc.org/ticket/2212#comment:8>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list