Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2659
---
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2659#discussion_r185929379
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/TopologyDetails.java ---
@@ -293,7 +295,7 @@ public Double getTotalMemReqTask(ExecutorDetails
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2659#discussion_r185929506
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -533,11 +533,11 @@ private static StormBase make(TopologyStatus status
GitHub user agresch opened a pull request:
https://github.com/apache/storm/pull/2659
STORM-3042 restore topology.acker.cpu.pcore.percent
TOPOLOGY_ACKER_CPU_PCORE_PERCENT and other similar settings in
adjustResourcesForExec() were orphaned accidentally by a previous pull request.
T