[ 
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

Reply via email to