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