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