Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2631
---
Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180811674
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/resource/normalization/NormalizedResourceRequest.java
---
@@ -187,4 +187,29 @@ public d
Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180800090
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/resource/normalization/NormalizedResourceRequest.java
---
@@ -187,4 +187,29 @@ public d
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180755909
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/Cluster.java ---
@@ -79,8 +81,8 @@
private Set blackListedHosts = new HashSet<>();
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180792829
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/resource/normalization/NormalizedResourceRequest.java
---
@@ -187,4 +187,29 @@ public double
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180767148
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/Cluster.java ---
@@ -301,17 +304,7 @@ public boolean needsSchedulingRas(TopologyDetails
topo
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2631#discussion_r180794169
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/resource/normalization/NormalizedResourceRequest.java
---
@@ -187,4 +187,29 @@ public double
GitHub user kishorvpatil opened a pull request:
https://github.com/apache/storm/pull/2631
[STORM-3025] Optimize Cluster methods with Caching to avoid loopy loops
Optimizing the `Cluster` methods that are called from with high frequency
calls to speed-up scheduling time on new topol