[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/916 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabl

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-162618653 Created [STORM-1377](https://issues.apache.org/jira/browse/STORM-1377) for nimbus-auth-test issues. Checks have passed, so I will merge this since there are binding +

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread d2r
Github user d2r closed the pull request at: https://github.com/apache/storm/pull/916 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread d2r
GitHub user d2r reopened a pull request: https://github.com/apache/storm/pull/916 [STORM-1364] logs the storm version when starting You can merge this pull request into a Git repository by running: $ git pull https://github.com/d2r/storm storm-1364-deamons-log-version-on-star

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-162593739 > 3 failures in nimbus-auth-test for storm-core on jdk8 only. Need to look at why it happened. I did look at these test failures. They are likely not related to this

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-07 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-162589790 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled an

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-06 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-162433815 It seems just add some log info --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-04 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161999062 3 failures in nimbus-auth-test for storm-core on jdk8 only. Need to look at why it happened. --- If your project is set up for it, you can reply to this email and have your

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread wuchong
Github user wuchong commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161851976 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread knusbaum
Github user knusbaum commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161800044 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161784128 +1 looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161782690 closing/reopening to trigger another ci build --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread d2r
GitHub user d2r reopened a pull request: https://github.com/apache/storm/pull/916 [STORM-1364] logs the storm version when starting You can merge this pull request into a Git repository by running: $ git pull https://github.com/d2r/storm storm-1364-deamons-log-version-on-star

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread d2r
Github user d2r closed the pull request at: https://github.com/apache/storm/pull/916 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161728319 ``` testPollWhileThereAreNoDataAvailable(backtype.storm.utils.ShellBoltMessageQueueTest) Time elapsed: 1.005 sec <<< FAILURE! junit.framework.AssertionFailedError:

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread unsleepy22
Github user unsleepy22 commented on the pull request: https://github.com/apache/storm/pull/916#issuecomment-161705726 LGTM +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enab

[GitHub] storm pull request: [STORM-1364] logs the storm version when start...

2015-12-03 Thread d2r
GitHub user d2r opened a pull request: https://github.com/apache/storm/pull/916 [STORM-1364] logs the storm version when starting You can merge this pull request into a Git repository by running: $ git pull https://github.com/d2r/storm storm-1364-deamons-log-version-on-start