rmatharu commented on a change in pull request #980: Cleaned up AM logs and
naming convention
URL: https://github.com/apache/samza/pull/980#discussion_r270500918
##########
File path:
samza-core/src/main/java/org/apache/samza/clustermanager/AbstractContainerAllocator.java
##########
@@ -121,56 +118,54 @@ public void run() {
}
/**
- * Assign resources from the cluster manager and matches them to run
container processes on them.
- *
+ * Assigns resources received from the cluster manager to processors.
*/
protected abstract void assignResourceRequests();
/**
- * Updates the request state and runs a container process on the specified
host. Assumes a resource
+ * Updates the request state and runs a processor on the specified host.
Assumes a resource
Review comment:
Assumed a container is available on the given preferredHost.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
