[GitHub] storm issue #2433: [STORM-2693] Heartbeats and assignments promotion for sto...

2018-03-12 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/2433 @revans2 Really thx, i will fix the nits problems as soon as possible, then i will merge this patch in. ---

[GitHub] storm pull request #2592: STORM-2993: Storm HDFS bolt throws ClosedChannelEx...

2018-03-12 Thread arunmahadevan
GitHub user arunmahadevan opened a pull request: https://github.com/apache/storm/pull/2592 STORM-2993: Storm HDFS bolt throws ClosedChannelException when Timed rotation policy is used The TimedRotation should synchronize so that the bolt does not attempt to write to a stale

[GitHub] storm issue #2433: [STORM-2693] Heartbeats and assignments promotion for sto...

2018-03-12 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2433 @danny0405 I created https://github.com/danny0405/storm/pull/2 to add in authorization support for the supervisor and the new nimbus APIs. ---

[GitHub] storm pull request #2591: STORM-2979

2018-03-12 Thread hummelm
GitHub user hummelm opened a pull request: https://github.com/apache/storm/pull/2591 STORM-2979 The pom modification is only for test propose. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hummelm/storm 1.0.6-p1-fix-STORM-2979

[GitHub] storm issue #459: avoiding rawtypes in IBold.prepare in order to allow usage...

2018-03-12 Thread knusbaum
Github user knusbaum commented on the issue: https://github.com/apache/storm/pull/459 Yes, I'll retracting my -1. Sorry for letting this one languish. ---

[GitHub] storm issue #2433: [STORM-2693] Heartbeats and assignments promotion for sto...

2018-03-12 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2433 OK I'll try to put up another pull request to cover basic auth for the supervisor. ---

[GitHub] storm issue #459: avoiding rawtypes in IBold.prepare in order to allow usage...

2018-03-12 Thread d2r
Github user d2r commented on the issue: https://github.com/apache/storm/pull/459 @krichter722, I am willing to disregard @knusbaum 's -1, but this PR needs to be updated as most of these files have been moved within the project. I checked a few of these, and it seems they