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
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.
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.
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.
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 to the
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 to the
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 to
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 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 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 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 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 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 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 is
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 user nilday commented on a diff in the pull request:
https://github.com/apache/storm/pull/1674#discussion_r93210117
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java
---
@@ -0,0 +1,252 @@
+/**
+ * Licensed to the Apache
Github user nilday commented on a diff in the pull request:
https://github.com/apache/storm/pull/1674#discussion_r93210078
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java
---
@@ -0,0 +1,252 @@
+/**
+ * Licensed to the Apache
Github user nilday commented on a diff in the pull request:
https://github.com/apache/storm/pull/1674#discussion_r93206374
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java
---
@@ -0,0 +1,252 @@
+/**
+ * Licensed to the Apache
Github user nilday commented on a diff in the pull request:
https://github.com/apache/storm/pull/1674#discussion_r93206136
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java
---
@@ -0,0 +1,252 @@
+/**
+ * Licensed to the Apache
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 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 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 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 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 bran
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 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 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 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 user nilday commented on the issue:
https://github.com/apache/storm/pull/1674
Thanks for all the advises you give. For the suggestions given by @revans2 :
1) We concerned implement blacklist in nimbus before. As a newbie in storm
contribution and clojure, I choose to
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
30 matches
Mail list logo