[
https://issues.apache.org/jira/browse/ZOOKEEPER-2358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15122330#comment-15122330
]
Ian Dimayuga commented on ZOOKEEPER-2358:
-----------------------------------------
Possibly. Right now they are using two different strategies due to the
different ways the test can access the internals. The existing test mocks the
NIO socket and selection which works for NIO, but it was easier to do the Netty
test end-to-end. The end-to-end test would work for NIO too, but it may not be
necessary.
> NettyServerCnxn leaks watches upon close
> ----------------------------------------
>
> Key: ZOOKEEPER-2358
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2358
> Project: ZooKeeper
> Issue Type: Bug
> Affects Versions: 3.4.7, 3.5.1
> Reporter: Ian Dimayuga
> Assignee: Ian Dimayuga
> Fix For: 3.4.8, 3.5.2
>
> Attachments: ZOOKEEPER-2358-3.4.patch, ZOOKEEPER-2358.patch
>
>
> NettyServerCnxn.close() neglects to call zkServer.removeCnxn the way
> NIOServerCnxn.close() does. Also, WatchLeakTest does not test watch leaks in
> Netty.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
