Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2685#discussion_r192583770
--- Diff:
storm-core/test/jvm/org/apache/storm/messaging/netty/NettyTest.java ---
@@ -0,0 +1,382 @@
+/*
+ * Copyright 2018 The Apache Software Foundatio
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2699
---
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2691
@revans2 Do you still plan to do manual tests on this patch? Sadly I
couldn't do it myself for now, so would like to rely on your test result.
---
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2685#discussion_r192579594
--- Diff:
storm-core/test/jvm/org/apache/storm/messaging/netty/NettyTest.java ---
@@ -0,0 +1,382 @@
+/*
+ * Copyright 2018 The Apache Software Fo
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2685#discussion_r192579513
--- Diff:
storm-core/test/jvm/org/apache/storm/messaging/netty/NettyTest.java ---
@@ -0,0 +1,382 @@
+/*
+ * Copyright 2018 The Apache Software Fo