BIND 10 #2207: define and implement (datasrc::memory::)ZoneUpdater class

BIND 10 Development do-not-reply at isc.org
Fri Oct 19 20:41:54 UTC 2012


#2207: define and implement (datasrc::memory::)ZoneUpdater class
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20121023
  medium                             |            Resolution:
                  Component:  data   |             Sensitive:  0
  source                             |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  7
            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:16 vorner]:

 The branch looks okay for merge (note: I made one final small cleanup).

 One non-blocking comment: if you mean by
 {{{#!cpp
     // For some reason it doesn't seem to work by the ZoneNode typedef,
 using
     // the full definition instead.
 }}}
 that this doesn't compile for you:
 {{{#!cpp
     const ZoneNode* node;
     EXPECT_EQ(ZoneTree::EXACTMATCH,
 }}}
 it worked here.  So, that's probably compiler dependent.

 > > > In that case, there's a confusion about `Unexpected`. Grep through
 the `memory/` directory for it and you'll see many occurrences of
 `Unexpected`, and none of them looks like a system error.
 > >
 > > I guess many of you were added by yourself:-) but in any case I admit
 > > it hasn't been officially clarified which exception (especially
 > > `Unexpected`) should be when.  We'll probably need a cleanup (and
 > > documentation) ticket.
 >
 > I'm not aware of me adding them, but I might have short memory. Will you
 create the clean-up ticket, please?

 Okay.

-- 
Ticket URL: <http://bind10.isc.org/ticket/2207#comment:17>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list