[ https://issues.apache.org/jira/browse/YARN-10212?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17078598#comment-17078598 ]
Jonathan Hung commented on YARN-10212: -------------------------------------- Thanks [~BilwaST], in general looks good, some minor style issues: * In TestResourceManager.java, can we change {{fail("Exception is expected because the global max attempts" +}} to {{fail("Exception is expected because AM max attempts" +}} * In YarnConfiguration.java: {{* an application,if unset by user.}} -> can we add a space after the comma * In yarn-default.xml, for theĀ comment forĀ yarn.resourcemanager.am.max-attempts: * {noformat} <description>The maximum number of application attempts. Each application master can specify its individual maximum number of application attempts via the API, but the individual number cannot be more than the global upper bound.This value is being set only if global max attempts is unset. The default number is set to 2, to {noformat} can we change this to * {noformat} <description>The default maximum number of application attempts, if unset by the user. Each application master can specify its individual maximum number of application attempts via the API, but the individual number cannot be more than the global upper bound in yarn.resourcemanager.am.global.max-attempts. The default number is set to 2, to{noformat} > Create separate configuration for max global AM attempts > -------------------------------------------------------- > > Key: YARN-10212 > URL: https://issues.apache.org/jira/browse/YARN-10212 > Project: Hadoop YARN > Issue Type: Improvement > Reporter: Jonathan Hung > Assignee: Bilwa S T > Priority: Major > Attachments: YARN-10212.001.patch, YARN-10212.002.patch, > YARN-10212.003.patch > > > Right now user's default max AM attempts is set to the same as global max AM > attempts: > {noformat} > int globalMaxAppAttempts = conf.getInt(YarnConfiguration.RM_AM_MAX_ATTEMPTS, > YarnConfiguration.DEFAULT_RM_AM_MAX_ATTEMPTS); {noformat} > If we want to increase global max AM attempts, it will also increase the > default. So we should create a separate global AM max attempts config to > separate the two. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org