Opened 8 years ago

Closed 7 years ago

#1767 closed task (duplicate)

revisit class organization of ZoneFinder::Context

Reported by: jinmei Owned by:
Priority: high Milestone:
Component: data source Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket: auth performance
Estimated Difficulty: 6 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description (last modified by jinmei)

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.

Subtickets

Change History (7)

comment:1 Changed 8 years ago by jinmei

  • Description modified (diff)

comment:2 Changed 8 years ago by jinmei

  • Summary changed from revist class organization of ZoneFinder::Context to revisit class organization of ZoneFinder::Context

comment:3 Changed 8 years ago by jelte

  • Estimated Difficulty changed from 0 to 6

comment:4 Changed 8 years ago by jinmei

  • Milestone changed from Year 3 Task Backlog to Sprint-20120403

comment:5 Changed 8 years ago by jinmei

  • Milestone changed from Sprint-20120403 to Next-Sprint-Proposed

(oops I meant to move it to the next sprint proposed queue. now doing so)

comment:6 Changed 7 years ago by jinmei

This issues was addressed in #2060. Closing this ticket.

comment:7 Changed 7 years ago by jinmei

  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.