BIND 10 #1332: Implement ZoneJournalReader class
BIND 10 Development
do-not-reply at isc.org
Tue Nov 15 07:48:56 UTC 2011
#1332: Implement ZoneJournalReader class
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: accepted
Type: task | Milestone:
Priority: major | Sprint-20111122
Component: data | Resolution:
source | Sensitive: 0
Keywords: | Sub-Project: DNS
Defect Severity: N/A | Estimated Difficulty: 5
Feature Depending on Ticket: | Total Hours: 0
Add Hours to Ticket: |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
trac1332 is ready for review.
It depends on #1330 (and on #1331 for writing tests), and uses
an intermediate versions of the correspoinding branches. Hopefully
the interfaces are fixed and we don't have to modify this branch
due to last minute changes to these branches. The first two commits
are for merging these branches and should be ignored.
I think the implementation itself is mostly straightforward. One thing
that may require a discussion would be how to return an error when
an unexpected input is given (non existing zone or serials). I chose
to return a result code rather than an exception because these event
are not really exceptional (an IXFR client can request a diff for
non existent version, and, depending on how we implement the application
non existent zone case can also happen at the level of this API), and
especially for C++ I believe it's better not to rely on exceptions for
normal events. But this could be a debatable point, and right now
it's not very consistent throughout the data source API, so we might
also have to discuss unifying the interface either way (but that level
of discussion will be a topic of a separate task/thread).
This is the proposed changelog entry:
{{{
317.? [func] jinmei
datasrc: Added C++ API for retrieving difference of two versions
of a zone. A new ZoneJournalReader class was introduced for this
purpose, and a corresponding factory method was added to
DataSourceClient.
(Trac #1332, git TBD)
}}}
--
Ticket URL: <http://bind10.isc.org/ticket/1332#comment:6>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list