BIND 10 #2209: define and implement ConfigurableClientList::getCacheZoneUpdater()

BIND 10 Development do-not-reply at isc.org
Thu Oct 25 19:10:22 UTC 2012


#2209: define and implement ConfigurableClientList::getCacheZoneUpdater()
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  UnAssigned
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20121106
  medium                             |            Resolution:
                  Component:  data   |             Sensitive:  0
  source                             |           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      |
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => UnAssigned
 * status:  accepted => reviewing


Comment:

 Hello

 The branch is ready for review. But there's a catch.

 It turned out the branch depended on #2208 (and I didn't know the ticket
 number
 when I hit the problem). So I added a „hack“ to get some zone table
 segment
 into the in-memory client in a92defd1deceebc13a7bcc93cee6579ed42be41e. I
 think
 #2208 will get merged soon, so once this branch is ready for merge, I'll
 remove
 the hack and adjust the rest. But I don't merge for now, since it would
 make it
 harder to review.

 Also, I made the reload tests templated and reused them to test the new
 method.
 Once the reload is no longer needed, we could un-template them again. Or
 we can
 keep reload, I turned it into a slim wrapper around the new method, so
 there's
 not much extra code.

 Also, there are two small unrelated clean ups. I could remove them if they
 were
 problematic, but I think at least one of them is an obvious and serious
 bug.

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


More information about the bind10-tickets mailing list