BIND 10 #3080: New DB backend: Postgres (patch)
BIND 10 Development
do-not-reply at isc.org
Fri Mar 14 18:56:14 UTC 2014
#3080: New DB backend: Postgres (patch)
-------------------------------------+-------------------------------------
Reporter: dclink | Owner:
Type: enhancement | UnAssigned
Priority: medium | Status:
Component: dhcpdb | reviewing
Keywords: | Milestone: DHCP-
Sensitive: 0 | Kea0.9-alpha
Sub-Project: DHCP | Resolution:
Estimated Difficulty: 3 | CVSS Scoring:
Total Hours: 0 | Defect Severity: N/A
| Feature Depending on Ticket: 2592
| Add Hours to Ticket: 0
| Internal?: 0
-------------------------------------+-------------------------------------
Changes (by tomek):
* owner: tomek => UnAssigned
* status: assigned => reviewing
Comment:
The code is now available for review. See trac3080.
I have extensively cleaned up the patch, added over 20 unit-tests, added
User's Guide sections, added Developer's Guide text and made several other
smaller improvements.
Please review.
Note that the code still requires some work, but I decided that it is
stable and clean for now to make its first attempt at review.
I think we'll need to consider (as additional tickets):
- using boost::lexical_cast<> instead of stringstream
- many parameters can be consts
- better line wrapping
- mode detailed comments for code in pgsql_lease_mgr.cc
--
Ticket URL: <http://bind10.isc.org/ticket/3080#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list