[ 
https://issues.apache.org/jira/browse/YARN-727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13703683#comment-13703683
 ] 

Xuan Gong commented on YARN-727:
--------------------------------

bq. No modifications to TestYarnClient even though YarnClient has changed?
YarnCLI seems to be using only one flavor of the api. The other api in 
YarnClient[Impl] not being tested?

No modifications to TestYarnClient, the api getApplications is tested in 
TestYarnCLI.
Not all the apis have been tested in TestYarnClient or TestYarnCLI.
Those apis have not been tested:
getQueueAclsInfo()
getChildQueueInfos()
getRootQueueInfos()
getAllQueues()
getQueueInfo()
getRMDelegationToken()
getYarnClusterMetrics()
createApplication()
                
> ClientRMProtocol.getAllApplications should accept ApplicationType as a 
> parameter
> --------------------------------------------------------------------------------
>
>                 Key: YARN-727
>                 URL: https://issues.apache.org/jira/browse/YARN-727
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 2.1.0-beta
>            Reporter: Siddharth Seth
>            Assignee: Xuan Gong
>            Priority: Blocker
>         Attachments: YARN-727.10.patch, YARN-727.11.patch, YARN-727.12.patch, 
> YARN-727.13.patch, YARN-727.14.patch, YARN-727.15.patch, YARN-727.16.patch, 
> YARN-727.17.patch, YARN-727.18.patch, YARN-727.19.patch, YARN-727.1.patch, 
> YARN-727.20.patch, YARN-727.2.patch, YARN-727.3.patch, YARN-727.4.patch, 
> YARN-727.5.patch, YARN-727.6.patch, YARN-727.7.patch, YARN-727.8.patch, 
> YARN-727.9.patch
>
>
> Now that an ApplicationType is registered on ApplicationSubmission, 
> getAllApplications should be able to use this string to query for a specific 
> application type.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to