BIND 10 #1332: Implement ZoneJournalReader class

[BIND 10 Development]

#1332: Implement ZoneJournalReader class
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       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):

 Replying to [comment:13 vorner]:

 > > >  * The database is expected to hold if a given diff is addition or
 removal. Why doesn't the interface provide this information? One comment
 says that an application might want to explicitly check that it is correct
 IXFR sequence, but it can't really do so without this information.
 > >
 > > Because it's basically part of the interface contracts (i.e., if it
 > > doesn't hold it's a bug of the underlying implementation or someone
 > > intentionally tweak the data by hand).  An applicatoin can check (if
 > > it really wants) the sequence is valid just like when xfrin checks the
 > > sequence is valid from an IXFR response (which doesn't contain an
 > > explicit flag of add or delete).
 >
 > No, I'm OK with the behaviour, considering that the only use we will
 have will probably be in IXFR-out anyway. It just didn't look too much
 consistent with the comment about checking. Maybe the comment could be
 neutralized slightly?

 For now I don't have a concrete idea of how to improve the wording,
 so I've merged the current branch anyway.  If you have a specific
 suggestion that will be appreicated.

 > Anyway, I think it can be merged (with or without a change to the
 comment, as you see fit).

 Okay, thanks for the review.  As noted above merge done.  Closing
 the ticket for now.

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