[
https://issues.apache.org/jira/browse/ZOOKEEPER-432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12719830#action_12719830
]
Mahadev konar commented on ZOOKEEPER-432:
-----------------------------------------
agreed. .. ill commit the patch without chmod_tests.patch ...
> Various improvements to zkpython bindings
> -----------------------------------------
>
> Key: ZOOKEEPER-432
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-432
> Project: Zookeeper
> Issue Type: Improvement
> Components: contrib-bindings
> Reporter: Henry Robinson
> Assignee: Henry Robinson
> Fix For: 3.2.0
>
> Attachments: ZOOKEEPER-432.patch, ZOOKEEPER-432.patch
>
>
> What's new, from the README for this new version:
> ----------------------------------
> More test coverage.
> Better reference counting, fixing at least two serious bugs.
> Out-of-range zhandles are now checked, fixing a potential security hole.
> Docstrings! Editing and cleanup required, but most of the text is there.
> zookeeper.set_watcher is now implemented correctly.
> zookeeper.client_id is now implemented correctly. zookeeper.init now respects
> the client_id parameter.
> get_context and set_context have been removed from the API. The context
> mechanism is used by PyZK to store the callables that are dispatched by
> C-side watchers. Messing with this from Python-side causes bugs very quickly.
> You should wrap all desired context up in a callable and then use
> zookeeper.set_watcher to attach it to the global watcher.
> Many methods now have optional parameters (usually if you can specify a
> watch, it's optional). The only time where genuinely optional parameters are
> still mandatory is when a required parameters comes after it. Currently we
> still respect the ZK C client parameter ordering. For example, you can simply
> connect with zookeeper.init("host:port") and ignore the other three
> parameters.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
