BIND 10 #2212: support "load zone" command in the configurator thread
BIND 10 Development
do-not-reply at isc.org
Fri Oct 26 21:12:48 UTC 2012
#2212: support "load zone" command in the configurator thread
-------------------------------------+-------------------------------------
Reporter: | Owner: jelte
jinmei | Status: reviewing
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):
Replying to [comment:8 jinmei]:
> 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.
Based on the latest status of #2209, I'll change this part as follows:
{{{#!diff
- case datasrc::ConfigurableClientList::ZONE_RELOADED: // XXX
misleading name
+ case datasrc::ConfigurableClientList::ZONE_SUCCESS:
}}}
(But I won't merge #2209 in this branch just for this, in order to
keep it easier to track. I'll just make this change on final merge -
if I forget it, the compiler will soon notice that).
--
Ticket URL: <http://bind10.isc.org/ticket/2212#comment:14>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list