BIND 10 #1198: Split DatabaseClient::Finder::find

BIND 10 Development do-not-reply at isc.org
Tue Nov 29 06:49:58 UTC 2011


#1198: Split DatabaseClient::Finder::find
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  vorner                             |                Status:  reviewing
                       Type:         |             Milestone:
  enhancement                        |  Sprint-20111206
                   Priority:  minor  |            Resolution:
                  Component:  data   |             Sensitive:  0
  source                             |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  4
            Defect Severity:  High   |           Total Hours:  0
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:15 stephen]:

 A couple of preliminary comments/counter questions:

 - Is change 717946a only a style fix of comments?  If so, I can
   concentrate on the essential changes and it would be quite helpful
   as the change in this commit is big.
 - About this:
 > There are four TODOs in database.cc: three are requesting clarification
 of the algorithm (in particular, reasons why it does/does not do
 something).  The fourth - line 792 - proposes alternate processing at a
 given condition.  Comments/thoughts requested.
   Are you asking me to clarify these TODOs to complete this ticket, or
   is this just a general note?  Especially in the case of the former,
   I'd wonder whether you are seeking answers to all four TODOs, since
   some of them seem to have been there before this branch started.

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


More information about the bind10-tickets mailing list