Opened 7 years ago

Closed 7 years ago

#2161 closed task (complete)

remove old datasrc API code

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


Things like:

  • lib/datasrc/data_source.{h,cc}
  • lib/datasrc/cache.{h,cc}
  • lib/datasrc/sqlite3_datasrc.{h,cc}
  • lib/datasrc/static_datasrc.{h,cc}

and their tests.

This is a prerequisite for further cleanups like #254 and #2153.


Change History (9)

comment:1 Changed 7 years ago by vorner

  • Milestone set to Next-Sprint-Proposed

I think that we're no longer using the old API, so it's about time to get rid of it, so we don't need to compile and maintain it.

comment:2 Changed 7 years ago by jinmei

  • Milestone set to Next-Sprint-Proposed

comment:3 Changed 7 years ago by muks

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

comment:4 Changed 7 years ago by vorner

  • Owner set to vorner
  • Status changed from new to accepted

comment:5 Changed 7 years ago by vorner

  • Owner changed from vorner to UnAssigned
  • Status changed from accepted to reviewing

I think this is done. I hope I didn't overlook anything. But one header is left
there ‒ there are some exceptions left. The rest could be just removed without
any changes anywhere. I did run all distchecks and lettuce tests and such, so
it should be safe.

I don't think this needs a changelog entry, there's no user-visible change.

comment:6 Changed 7 years ago by jelte

  • Owner changed from UnAssigned to jelte

comment:7 Changed 7 years ago by jelte

  • Owner changed from jelte to vorner

I see no big problems with this branch, and think it can be merged.

We may want to consider renaming the clients again, now that some names have been freed up (but not in this ticket).

comment:8 Changed 7 years ago by vorner

  • Total Hours changed from 0 to 2.31

Thank you, closing

comment:9 Changed 7 years ago by vorner

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