jnioche commented on PR #8416:
URL: https://github.com/apache/storm/pull/8416#issuecomment-3989478603
Sorry, had rushed this one a bit. About to push a fix
Root cause of the test failures: Both tests were accidentally relying on
the silent exception swallow. They called prepare() without any ZooKeeper in
the config, which caused mkStormClusterState to throw — but the old
e.printStackTrace() swallowed it and let the tests proceed with
stormClusterState = null. The tests happened to pass because neither test's
assertions touched cluster state.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
