BIND 10 #144: Review: Xfrout

[BIND 10 Development]

#144: Review: Xfrout
--------------------------+-------------------------------------------------
 Reporter:  zhanglikun    |        Owner:  zhanglikun                                       
     Type:  task          |       Status:  assigned                                         
 Priority:  major         |    Milestone:  02. Running, functional authoritative-only server
Component:  Unclassified  |   Resolution:                                                   
 Keywords:                |    Sensitive:  0                                                
--------------------------+-------------------------------------------------
Changes (by jinmei):

  * owner:  jinmei => zhanglikun


Comment:

 Replying to [ticket:144 zhanglikun]:
 >    Hi Jinmei, I plan to assign this ticket to you, then let you decide
 whether this feature should be included in the release on 4.19, or left
 the decision to jeremy .
 >    I know the code will be changed later since it doesn't support IXFR
 now.

 From a quick glance there isn't much documentation.

  - at least there should be overall design description of this module
  - any class and public function/method should have their description
  - any other non trivial design choices (if any) should also be
 documented.

 Regarding tests, do they check pathological cases (bogus request, sudden
 reset, any other cases that trigger an exception, etc) in addition to
 normal transactions?

 Please make sure these prerequisites are met, then ask fore review again.
 At the very least documentation should be much improved so I'm giving it
 back to you for now.

-- 
Ticket URL: <https://bind10.isc.org/ticket/144#comment:2>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development