[
https://issues.apache.org/jira/browse/ZOOKEEPER-1318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13265525#comment-13265525
]
Henry Robinson commented on ZOOKEEPER-1318:
-------------------------------------------
Although a test would be ideal, as I mentioned above:
* SessionExpired tests are tricky to write
* This patch is trivial
* We don't have coverage of this type ('does the correct exception get thrown')
in most other places for zkpython
> In Python binding, get_children (and get and exists, and probably others)
> with expired session doesn't raise exception properly
> -------------------------------------------------------------------------------------------------------------------------------
>
> Key: ZOOKEEPER-1318
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1318
> Project: ZooKeeper
> Issue Type: Bug
> Components: contrib-bindings
> Affects Versions: 3.3.3
> Environment: Mac OS X (at least)
> Reporter: Jim Fulton
> Assignee: Henry Robinson
> Fix For: 3.3.6, 3.4.4, 3.5.0
>
> Attachments: ZOOKEEPER-1318.patch, ZOOKEEPER-1318.patch
>
>
> In Python binding, get_children (and get and exists, and probably others)
> with expired session doesn't raise exception properly.
> >>> zookeeper.state(h)
> -112
> >>> zookeeper.get_children(h, '/')
> Traceback (most recent call last):
> File "<console>", line 1, in <module>
> SystemError: error return without exception set
> Let me know if you'd like me to work on a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
