Stig Døssing

@liucijus I get a 404 on your link, maybe the repo is private? Just for reference, I took a look at how the Gradle and Maven integrations handle this: The...

I think it would be helpful to have a way to specify that Awaitility should throw AssertionError instead of ConditionTimeoutException. It would make using Awaitility much less awkward in cases...

Hi @tjqiulu 0.9.x is no longer maintained. Please upgrade to a current version. The code you are pointing to still exists in master https://github.com/apache/storm/blob/1976cb332bc6e8dceaec278fb1c3927627981081/storm-webapp/src/main/java/org/apache/storm/daemon/ui/WEB-INF/js/script.js#L88. I believe it is a false...

The changes look good to me. Please raise an issue at https://issues.apache.org/jira and rename this PR and the commit message to contain the issue number.

@bd2019us Thanks, looks great. Could you squash to one commit, and we can merge after the 24-hour waiting period?

@wxy929629 Thanks. Could you fix the checkstyle issues that have been introduced? The build output has the following lines ``` [WARN] /home/travis/build/apache/storm/external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java:147:86: WhitespaceAround: '{' is not preceded with whitespace. [WhitespaceAround]...

Hi, thanks for the patch. Upgrading the eventhubs client sounds good, but we need this to go in the newer branches first. The fix needs to go in master, and...

I would expect any API changes to be very minor. As far as I recall, the only API change you're likely to see as an implementer of a spout is...

@Ethanlm I'm not sure. I think we should probably add the field for all bolts and classes serialized as part of bolt state, since these classes are likely to be...

The exit code is 20. Try looking for uses of `Utils.exitProcess` with `20` as the argument. If you change the values to be unique and rerun the tests, you should...