BIND 10 #221: review: Refactoring auth server and merge the axfr and notify logic into it (was: Refactoring auth server and merge the axfr and notify logic into it)

[BIND 10 Development]

#221: review: Refactoring auth server and merge the axfr and notify logic into it
-------------------------+--------------------------------------------------
 Reporter:  hanfeng      |        Owner:  UnAssigned                                    
     Type:  enhancement  |       Status:  reviewing                                     
 Priority:  major        |    Milestone:  05. 3rd Incremental Release: Serious Secondary
Component:  b10-auth     |   Resolution:                                                
 Keywords:               |    Sensitive:  0                                             
-------------------------+--------------------------------------------------
Changes (by jinmei):

  * owner:  hanfeng => UnAssigned


Comment:

 Replying to [comment:18 hanfeng]:

 > > What's the current status of this ticket?
 > >
 > > I'd like to see this feature in the next release (but I don't like to
 make an easy compromise on test coverage and code/documentation quality).
 If it stalls I'm happy to help write tests, etc.
 >
 > I am a little busy now, so if you can help write some tests, that will
 be great, thanks a lot

 I've added all possible tests I can think of, and fixed all bugs that were
 identified by the tests as well as the issues I've pointed out.

 There are 16 new (including one substantially revised) tests: 5 for AXFR
 and 11 for notify.  Please expect I'd request at least this level and
 amount of tests next time you ask for reivew:-)

 Now I'm rather a (co)author of this branch than a reviewer, with a large
 set of changes I made myself, and I'm afraid we need a different reviewer.
 I'll get this ticket unassigned.

 It would be great if we could get it reviewed and merged to trunk before
 the next week, but since it's a quite big change maybe it's too
 optimistic.  In any case, I think we should learn a less from this ticket:
 this is an example case how we tend to underestimate the workload of
 adding tests in planning.

-- 
Ticket URL: <http://bind10.isc.org/ticket/221#comment:19>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development