BIND 10 #1767: revist class organization of ZoneFinder::Context

[BIND 10 Development]

#1767: revist class organization of ZoneFinder::Context
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:
  jinmei                             |                Status:  new
                       Type:  task   |             Milestone:  Next-Sprint-
                   Priority:  high   |  Proposed
                  Component:  data   |            Resolution:
  source                             |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  0
Feature Depending on Ticket:  auth   |           Total Hours:  0
  performance                        |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Description changed by jinmei:

Old description:

> See http://bind10.isc.org/ticket/1607#comment:14
>
> We may want to introduce a pure abstract base class (that doesn't have
> any member variable) of `Context`: The in-memory version would be
> inherited from it directly and optimize everything; The default
> version would be another directly derived class of the base class.
> With this applications cannot directly get access to `rrset`, etc,
> so we'll need accessor virtual functions.
>
> We may also want to unify the interface of find() and findAll() (even
> if we still keep having two methods for the different purposes).  With
> the introduction of the accessor mentioned above it may be easier.
>
> The plan is to work on this ticket with or immediately after #1608,
> #1753 and #1747 as we'll get more concrete idea of what we want this
> base class to become.

New description:

 See http://bind10.isc.org/ticket/1607#comment:14

 We may want to introduce a pure abstract base class (that doesn't have
 any member variable) of `Context`: The in-memory version would be
 inherited from it directly and optimize everything; The default
 version would be another directly derived class of the base class.
 With this applications cannot directly get access to `rrset`, etc,
 so we'll need accessor virtual functions.

 We may also want to unify the interface of find() and findAll() (even
 if we still keep having two methods for the different purposes).  With
 the introduction of the accessor mentioned above it may be easier.

 The plan is to work on this ticket with or immediately after #1608,
 #1753 and #1747 as we'll get more concrete idea of what we want this
 base class to become.

 After this work is done we should recheck the performance effect of
 #1753.

--

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