BIND 10 #1068: support writability in the new data source API
BIND 10 Development
do-not-reply at isc.org
Fri Aug 19 09:26:49 UTC 2011
#1068: support writability in the new data source API
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: accepted
Type: task | Milestone:
Priority: major | Sprint-20110830
Component: data | Resolution:
source | Sensitive: 0
Keywords: | Sub-Project: DNS
Defect Severity: N/A | Estimated Difficulty: 7.0
Feature Depending on Ticket: | Total Hours: 0
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
The first part of this taks is ready for review in the
trac1068review1 branch. It contains implementation of update related
methods of SQLite3Accessor (renamed from SQLite3Database, which is
part of refactoring change in this branch).
This branch also merged trac1183 as it seems to be the future
of the base code.
Some other notes:
- as noted above, there are some refactoring changes in this branch
that are not directly related to the topic of this task. First is to
rename "database" to "accessor" in more places of the code. The
other is to update/cleanup the internal details of
sqlite3_accessor.cc by unifying SQL statements in SQLite3Parameters.
IMO this will improve the readability quite a lot. I also removed
commented out unused statements. We can always add them as we see
the need for it.
- there is an important bug fix that is actually in the master:
getZone() can leave an unfinished statement. Assuming this branch
can be merged not so long future, I'd rather fix it within this
branch. Right now it's not used in any application, so it should be
okay.
- AddRecordColumns and DeleteRecordParams are almost unused right now.
They will be used in the second half of the task.
--
Ticket URL: <http://bind10.isc.org/ticket/1068#comment:4>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list