[ https://issues.apache.org/jira/browse/YARN-10172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17068619#comment-17068619 ]
Bibin Chundatt commented on YARN-10172: --------------------------------------- [~cyrusjackson25] Please check the checkstyle issues.Apart from that changes looks good to me. {code} 184 String DEFAULT_APPLICATION_PLACEMENT_TYPE_CLASS = "org.apache.hadoop.yarn." 185 + "server.resourcemanager.scheduler.capacity." 186 + "yarnpp.YarnppLocalityAppPlacementAllocator"; {code} # Rename YarnppLocalityAppPlacementAllocator -> DummyLocalityAppPlacementAllocator # The package name also could be short. [~sunil.gov...@gmail.com] Would you like take a look > Default ApplicationPlacementType class should be configurable > ------------------------------------------------------------- > > Key: YARN-10172 > URL: https://issues.apache.org/jira/browse/YARN-10172 > Project: Hadoop YARN > Issue Type: Improvement > Reporter: Cyrus Jackson > Assignee: Cyrus Jackson > Priority: Minor > Attachments: YARN-10172.001.patch > > > This can be useful in scheduling apps based on the configured placement type > class rather than resorting to LocalityAppPlacementAllocator -- 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