Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2385#discussion_r149162901
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/TopologyDetails.java ---
@@ -289,17 +300,13 @@ public Double getTotalMemReqTask(ExecutorDetails
exec) {
/**
* Gets the total memory resource list for a set of tasks that is part
of a topology.
- * @return Map<ExecutorDetails, Double> a map of the total memory
requirement for all tasks in topology topoId.
+ * @param executors
+ * @return Map<ExecutorDetails, Double> ,
--- End diff --
The type is not needed here, and I think it all works on a single line.
---
