BIND 10 #2100: Update ZoneTable class
BIND 10 Development
do-not-reply at isc.org
Thu Aug 9 01:06:31 UTC 2012
#2100: Update ZoneTable class
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: accepted
Type: task | Milestone:
Priority: | Sprint-20120821
medium | Resolution:
Component: data | Sensitive: 0
source | Sub-Project: DNS
Keywords: | Estimated Difficulty: 4
Defect Severity: N/A | Total Hours: 0
Feature Depending on Ticket: |
scalable inmemory |
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
trac2100 is ready for review.
In the first commit (57de755) I copied the old implementation. This
commit can be ignored for review. But it may be easier to read the
entire implementation rather than taking the diff from the initial
commit. It's not that big.
I've added a mostly empty implementation of `ZoneData` (without direct
for this class at the moment). At this moment it's mostly a
placeholder of the zone table.
On looking into the details, I decided to change the return type of
findZone(); instead of creating a zone finder here, it now just
returns the underlying `ZoneData` and let the caller to use it (e.g.
for creating a finder containing the data). It seemed cleaner and
will make it easier to use the zone table for different purposes (like
creating an updater).
The last two commits are not directly related to the subject of this
branch, but I thought they're necessary and should be simple enough.
See the commit logs. If these happen to be controversial it's okay
for me to exclude them in this ticket.
--
Ticket URL: <http://bind10.isc.org/ticket/2100#comment:6>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list