BIND 10 #1607: define ZoneFinder::findAdditional() and implement its default version

BIND 10 Development do-not-reply at isc.org
Fri Mar 2 09:34:48 UTC 2012


#1607: define ZoneFinder::findAdditional() and implement its default version
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  accepted
                       Type:  task   |             Milestone:
                   Priority:  major  |  Sprint-20120306
                  Component:         |            Resolution:
  b10-auth                           |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  4
Feature Depending on Ticket:  auth   |           Total Hours:  0
  performance                        |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 trac1607 is ready for review.

 I've extended the scope of this ticket as discussed on the dev list.
 See the one with the subject of "proposal: a performance extendable
 auth query logic framework".

 Unfortunately the amount of diff is big (again...), but hopefully it's
 not as bad as it might look; most part of the changes are
 straightforward refactoring due to the interface change.

 - the first two commits are relatively straightforward refactoring but
   the diff is big, so it's probably better to review these separately.
 - 118f1f9 and ac29935 are unrelated extension for test utility and can
   be reviewed separately.
 - from 4bcec5f to 1633572 (excluding 118f1f9 and ac29935) are the main
   changes for this branch.
 - 29fab38 and 6f31530 are updates to the auth::Query class due to the
   interface change.  I believe these are straightforward.
 - 64611b9 is an unrelated optimization.  If it looks too out-of-scope
   I'm okay with excluding it.

 This is the proposed changelog entry:
 {{{
 392.?   [func]*         jinmei
         libdatasrc: change the return type of ZoneFinder::find() so it can
         contain more context of the search, which can be used for
         optimizing post find() processing.  A new method getAdditional()
         is added to it for finding additional RRsets based on the result
         of find().  External behavior shouldn't change.  The query
         handling code of b10-auth now uses the new interface.
         (Trac #1607, git TBD)
 }}}

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


More information about the bind10-tickets mailing list