Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STORM-2918 Update Netty version #2541

Merged
merged 1 commit into from Feb 2, 2018
Merged

STORM-2918 Update Netty version #2541

merged 1 commit into from Feb 2, 2018

Conversation

@dbist
Copy link

dbist commented Jan 31, 2018

Ran mvn clean install -DskipTests on the parent project
ran mvn clean install on storm-core

got the following error, is the test flaky? Build ran successfully

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.apache.storm.serialization.SerializationTest
Running org.apache.storm.command.TestCLI
Running org.apache.storm.command.RebalanceTest
Running org.apache.storm.command.SetLogLevelTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.005 sec - in org.apache.storm.command.RebalanceTest
Running org.apache.storm.stats.TestStatsUtil
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.056 sec - in org.apache.storm.command.SetLogLevelTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.634 sec - in org.apache.storm.command.TestCLI
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.723 sec - in org.apache.storm.stats.TestStatsUtil
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.888 sec - in org.apache.storm.serialization.SerializationTest

Results :

Tests run: 15, Failures: 0, Errors: 0, Skipped: 0

[INFO]
[INFO] --- clojure-maven-plugin:1.7.1:test (test-clojure) @ storm-core ---
3784 [main] WARN  o.a.s.v.ConfigValidation - storm.messaging.netty.max_retries is a deprecated config please see class org.apache.storm.Config.STORM_MESSAGING_NETTY_MAX_RETRIES for more information.
Running org.apache.storm.drpc-test
Tests run: 7, Passed: 14, Failures: 0, Errors: 0
Running org.apache.storm.transactional-test
Tests run: 6, Passed: 108, Failures: 0, Errors: 0
Running org.apache.storm.scheduler-test
Tests run: 4, Passed: 64, Failures: 0, Errors: 0
Running org.apache.storm.messaging.netty-unit-test
Tests run: 5, Passed: 24, Failures: 0, Errors: 0
Running org.apache.storm.trident.tuple-test
Tests run: 6, Passed: 36, Failures: 0, Errors: 0
Running org.apache.storm.serialization-test
Tests run: 1, Passed: 0, Failures: 0, Errors: 0
Running org.apache.storm.security.auth.nimbus-auth-test
Tests run: 5, Passed: 39, Failures: 0, Errors: 0
Running org.apache.storm.security.auth.auth-test
Tests run: 14, Passed: 100, Failures: 0, Errors: 0
Running org.apache.storm.nimbus-test
162903 [main-SendThread(localhost:2181)] WARN  o.a.z.ClientCnxn - Session 0x0 for server null, unexpected error, closing socket connection and attempting reconnect
java.net.ConnectException: Connection refused
	at sun.nio.ch.SocketChannelImpl.checkConnect(Native Method) ~[?:1.8.0_162]
	at sun.nio.ch.SocketChannelImpl.finishConnect(SocketChannelImpl.java:717) ~[?:1.8.0_162]
	at org.apache.zookeeper.ClientCnxnSocketNIO.doTransport(ClientCnxnSocketNIO.java:361) ~[zookeeper-3.4.6.jar:3.4.6-1569965]
	at org.apache.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:1081) [zookeeper-3.4.6.jar:3.4.6-1569965]
Tests run: 41, Passed: 773, Failures: 0, Errors: 0
Running org.apache.storm.trident.state-test
Tests run: 5, Passed: 37, Failures: 0, Errors: 0
Running org.apache.storm.submitter-test
Tests run: 1, Passed: 13, Failures: 0, Errors: 0
Running org.apache.storm.grouping-test
Tests run: 3, Passed: 7, Failures: 0, Errors: 0
Running org.apache.storm.scheduler.multitenant-scheduler-test
Tests run: 17, Passed: 233, Failures: 0, Errors: 0
Running org.apache.storm.security.auth.auto-login-module-test
Tests run: 6, Passed: 22, Failures: 0, Errors: 0
Running org.apache.storm.versioned-store-test
Tests run: 2, Passed: 5, Failures: 0, Errors: 0
Running org.apache.storm.messaging.netty-integration-test
Tests run: 1, Passed: 1, Failures: 0, Errors: 0
Running org.apache.storm.metrics-test
Tests run: 8, Passed: 66, Failures: 0, Errors: 0
Running org.apache.storm.cluster-test
Tests run: 10, Passed: 92, Failures: 0, Errors: 0
Artem Ervits
@HeartSaVioR

This comment has been minimized.

Copy link
Contributor

HeartSaVioR commented Feb 1, 2018

Yes build is flaky and some of us are putting valuable efforts to fix it.

Copy link
Contributor

HeartSaVioR left a comment

Built binary and ran RollingTopWords. Looks good.
+1

We need to port back to 1.x version lines as well.

@ptgoetz

This comment has been minimized.

Copy link
Member

ptgoetz commented Feb 1, 2018

+1

@asfgit asfgit merged commit 956ba1c into apache:master Feb 2, 2018
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.