[
https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057148#comment-15057148
]
ASF GitHub Bot commented on STORM-1038:
---------------------------------------
Github user rfarivar commented on a diff in the pull request:
https://github.com/apache/storm/pull/728#discussion_r47586860
--- Diff: storm-core/src/jvm/backtype/storm/messaging/netty/Client.java ---
@@ -182,7 +177,7 @@ private boolean connectionEstablished(Channel channel) {
// See:
// -
http://netty.io/3.9/api/org/jboss/netty/channel/ChannelEvent.html
// -
http://stackoverflow.com/questions/13356622/what-are-the-netty-channel-state-transitions
- return channel != null && channel.isConnected();
+ return channel != null && channel.isOpen();
--- End diff --
Why the change from connected to open? the channel could be open, but not
yet in the connected state, and this could cause a bug.
Unless Netty 4 has changed the model....
> Upgrade netty transport from 3.x to 4.x
> ---------------------------------------
>
> Key: STORM-1038
> URL: https://issues.apache.org/jira/browse/STORM-1038
> Project: Apache Storm
> Issue Type: Dependency upgrade
> Components: storm-core
> Reporter: Hang Sun
> Priority: Minor
> Labels: performance
> Original Estimate: 168h
> Remaining Estimate: 168h
>
> It will be nice to upgrade netty to 4.x to take advantage of its more
> efficient memory usage.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
