[ https://issues.apache.org/jira/browse/YARN-6883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16122063#comment-16122063 ]
Larry Lo commented on YARN-6883: -------------------------------- It looks like the same issue to change from if/else=if to switch statement in the same Class but different methods. Should I keep to fix this issue or something else? > AllocationFileLoaderService.reloadAllocations() should use a switch statement > in the main tag parsing loop instead of the if/else-if/... > ---------------------------------------------------------------------------------------------------------------------------------------- > > Key: YARN-6883 > URL: https://issues.apache.org/jira/browse/YARN-6883 > Project: Hadoop YARN > Issue Type: Improvement > Components: fairscheduler > Affects Versions: 3.0.0-alpha4 > Reporter: Daniel Templeton > Assignee: Larry Lo > Priority: Minor > Labels: newbie > > {code} if ("queue".equals(element.getTagName()) || > "pool".equals(element.getTagName())) { > queueElements.add(element); > } else if ("user".equals(element.getTagName())) { > ...{code} -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org