bipinprasad commented on a change in pull request #3328:
URL: https://github.com/apache/storm/pull/3328#discussion_r490617198
##########
File path:
storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/IStrategy.java
##########
@@ -19,21 +19,29 @@
/**
* An interface to for implementing different scheduling strategies for the
resource aware scheduling.
+ * Scheduler should call {@link #prepare(Map)} followed by {@link
#schedule(Cluster, TopologyDetails)}.
+ * <p>
+ * A fully functioning implementation is in the abstract class {@link
BaseResourceAwareStrategy}.
+ * Subclasses classes should extend {@link
BaseResourceAwareStrategy#BaseResourceAwareStrategy(boolean,
ObjectResourceSortType)}
Review comment:
Fixed documentation.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
