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