Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2385#discussion_r149162982 --- 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 --- End diff -- nit Having a param with no explanation is not really helpful.
---