[GitHub] storm issue #1674: STORM-2083: Blacklist scheduler

2017-06-28 Thread nilday
Github user nilday commented on the issue: https://github.com/apache/storm/pull/1674 @HeartSaVioR I'll rebase it as soon as I can. As the storm-core has been restructured, it may take a while. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm pull request #1965: STORM-2380 worker.childopts with whitespace inside...

2017-02-24 Thread nilday
GitHub user nilday opened a pull request: https://github.com/apache/storm/pull/1965 STORM-2380 worker.childopts with whitespace inside one param This pattern matches 1.everything starts with -XX:\w+ or -D[\w.]+ and followed with quoted (both ' and ") strings 2.every

[GitHub] storm pull request #1964: STORM-2380 worker.childopts with whitespace inside...

2017-02-24 Thread nilday
GitHub user nilday opened a pull request: https://github.com/apache/storm/pull/1964 STORM-2380 worker.childopts with whitespace inside one param This pattern matches 1.everything starts with -XX:\w+ or -D[\w.]+ and followed with quoted (both ' and ") strings 2.every

[GitHub] storm pull request #1963: STORM-2380 worker.childopts with whitespace inside...

2017-02-24 Thread nilday
GitHub user nilday opened a pull request: https://github.com/apache/storm/pull/1963 STORM-2380 worker.childopts with whitespace inside one param This pattern matches 1.everything starts with -XX:\w+ or -D[\w.]+ and followed with quoted (both ' and ") strings 2.every

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95340409 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/strategies/IBlacklistStrategy.java --- @@ -0,0 +1,37 @@ +/** + * Licensed

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95340338 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/strategies/IBlacklistStrategy.java --- @@ -0,0 +1,37 @@ +/** + * Licensed

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95340314 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/strategies/DefaultBlacklistStrategy.java --- @@ -0,0 +1,149 @@ +/** + * Licensed

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95339881 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,245 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95339408 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/CircularBuffer.java --- @@ -0,0 +1,174 @@ +/** + * Licensed to the Apache Software

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95339494 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/CircularBuffer.java --- @@ -0,0 +1,174 @@ +/** + * Licensed to the Apache Software

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95338774 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,245 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2017-01-10 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r95338650 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,245 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-12-20 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r93364833 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,252 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1836: Merge pull request #2 from apache/master

2016-12-20 Thread nilday
Github user nilday closed the pull request at: https://github.com/apache/storm/pull/1836 --- 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

[GitHub] storm pull request #1836: Merge pull request #2 from apache/master

2016-12-20 Thread nilday
GitHub user nilday opened a pull request: https://github.com/apache/storm/pull/1836 Merge pull request #2 from apache/master merge from origin You can merge this pull request into a Git repository by running: $ git pull https://github.com/nilday/storm master Alternatively you

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-12-20 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r93205875 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,252 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-12-20 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r93205840 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,252 @@ +/** + * Licensed to the Apache

[GitHub] storm issue #1674: STORM-2083: Blacklist scheduler

2016-12-08 Thread nilday
Github user nilday commented on the issue: https://github.com/apache/storm/pull/1674 As soon as I find out that nimbus has been transferred to Java, I start to work on this again. The blacklist scheduler has been working on our production environment(Storm v1.0.1) for several months

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-09-28 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r80849475 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,212 @@ +package

[GitHub] storm issue #1674: STORM-2083: Blacklist scheduler

2016-09-27 Thread nilday
Github user nilday commented on the issue: https://github.com/apache/storm/pull/1674 Thanks for you all guys. I will consider seriously all your suggestions. But currently I'll be still working on 1.0.x branch ,since my company has just update to 1.0 version and master branch

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-09-27 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r80839269 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,212 @@ +package

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-09-27 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r80839265 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,212 @@ +package

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-09-27 Thread nilday
Github user nilday commented on a diff in the pull request: https://github.com/apache/storm/pull/1674#discussion_r80839227 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java --- @@ -0,0 +1,212 @@ +package

[GitHub] storm issue #1674: STORM-2083: Blacklist scheduler

2016-09-22 Thread nilday
Github user nilday commented on the issue: https://github.com/apache/storm/pull/1674 @HeartSaVioR not yet, busy with something else. Will do it as soon as I have some time. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] storm pull request #1674: STORM-2083: Blacklist scheduler

2016-09-06 Thread nilday
GitHub user nilday opened a pull request: https://github.com/apache/storm/pull/1674 STORM-2083: Blacklist scheduler A resolve of [STORM-2083](https://issues.apache.org/jira/browse/STORM-2083) You can merge this pull request into a Git repository by running: $ git pull https