BIND 10 #137: review: some cleanups for xfrin

BIND 10 Development do-not-reply at isc.org
Wed Apr 7 07:34:30 UTC 2010


#137: review: some cleanups for xfrin
--------------------------+-------------------------------------------------
 Reporter:  jinmei        |       Owner:  zhanglikun
     Type:  defect        |      Status:  new       
 Priority:  major         |   Milestone:            
Component:  Unclassified  |    Keywords:            
Sensitive:  0             |  
--------------------------+-------------------------------------------------
 Please someone review r1684.  As noted in the commit log, this eliminated
 code redundancy due to AXFRInDB (btw this class name wasn't good because
 it had nothing to with (A)XFR in itself).

 The behavior should be generally kept, but failure mode can be different,
 especially when sqlite3.load() raises an exception.  But I suspect the
 axfrin code should generally be naive about exception handling and this
 should be revisited anyway.

 I'm not sure who's the best person as a reviewer, but giving it to likun
 as an initial guess.

-- 
Ticket URL: <https://bind10.isc.org/ticket/137>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list