hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wangda Tan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-4889) Changes in AMRMClient for identifying resource-requests explicitly
Date Tue, 23 Aug 2016 19:58:20 GMT

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

Wangda Tan commented on YARN-4889:

Thanks [~asuresh], some minor comments:

1) AMRMClient
- Annonate newly added constructors to @Public @Evolving (or @Unstable)? Suggest to use evolving
to make it consistent with annotations in ResourceRequest
- Why previous return values of getMatchingRequests is List<Collection> but the newly
added getMatchingRequests is Collection? Why previous return value is List of Collection?

2) AMRMClientImpl: 
- It’s better to make this to be INFO:
LOG.debug("No remoteRequestTable found with allocationRequestId="
 + ... )

3) Tests:
- {{testAllocRequestId}} has some duplicated logic from {{testAllocation}}, possible to merge

> Changes in AMRMClient for identifying resource-requests explicitly
> ------------------------------------------------------------------
>                 Key: YARN-4889
>                 URL: https://issues.apache.org/jira/browse/YARN-4889
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Subru Krishnan
>            Assignee: Arun Suresh
>         Attachments: YARN-4889.001.patch, YARN-4889.002.patch, YARN-4889.003.patch
> YARN-4879 proposes the notion of identifying allocate requests explicitly.. This JIRA
is to track the changes in AMRMClient to keep it wire compatible with the changes. Please
refer to the design doc in the parent JIRA for details.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org

View raw message