[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12831206#action_12831206
 ] 

Mahadev konar commented on ZOOKEEPER-579:
-----------------------------------------

+1 the patch looks good. ll run zkpython tests and post the results here.

henry, you might want to start adding comments to the zookeeper.c file so that 
its easier for other developers to work on it in case someone wants to pick it 
up. 



> zkpython needs more test coverage for ACL code paths
> ----------------------------------------------------
>
>                 Key: ZOOKEEPER-579
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-579
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: contrib-bindings
>    Affects Versions: 3.2.1
>            Reporter: Henry Robinson
>            Assignee: Henry Robinson
>         Attachments: zookeeper-579.patch
>
>
> zkpython's tests don't do a good enough job of exercising the ACL code paths. 
> A few new tests that confirm that setACL and friends are working correctly 
> are needed. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to