Github user serranom commented on a diff in the pull request:
https://github.com/apache/twill/pull/23#discussion_r98083899
--- Diff:
twill-yarn/src/main/java/org/apache/twill/internal/appmaster/RunningContainers.java
---
@@ -113,9 +117,11 @@ public Integer apply(BitSet input) {
private final Location applicationLocation;
private final Set<String> runnableNames;
private final Map<String, Map<String, String>> logLevels;
+ private final Map<String, Integer> maxRetries;
--- End diff --
from my analysis the association of instance ids does not necessarily
correspond to specific processes. it looked like there is a pool of requests,
a new request being serviced gets the lowest instance id and a failed request
gets put back on the queue. this is why i went with an instance adjusted
number of retries. did i miss something?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
