BIND 10 #292: Asynchronous support for python config ccsession library

[BIND 10 Development]

#292: Asynchronous support for python config ccsession library
--------------------------------+-------------------------------------------
      Reporter:  fujiwara       |        Owner:  fujiwara
          Type:  enhancement    |       Status:  assigned
      Priority:  minor          |    Milestone:          
     Component:  configuration  |   Resolution:          
      Keywords:                 |    Sensitive:  0       
Estimatedhours:  0              |        Hours:  0       
      Billable:  1              |   Totalhours:  0       
      Internal:  0              |  
--------------------------------+-------------------------------------------
Changes (by jelte):

  * owner:  jelte => fujiwara
  * status:  new => assigned


Comment:

 r2218 looks fine to me, apart from that there is no docstring for the new
 function

 """Parse the given message to see if there is a command or a configuration
 update. Calls the corresponding handler functions if present. Responds on
 the channel if the handler returns a message."""

 and perhaps the first function should have a comment that it calls this
 one if it reads a message from the channel

 I'm wondering though, should we make the new function return something so
 that the caller knows whether it did something (for instance, if the
 message did indeed have a command or config update, and it is handled,
 return true, or false otherwise)?

 BTW, slightly offtopic, but the experiments/fujiwara-stats branch does not
 compile for me, it looks like it's missing a Makefile in src/bin/stats/
 (I'm guessing it's there on your system but not added to svn)

 do you want to merge this separately from the rest of the branch?

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