[ https://issues.apache.org/jira/browse/YARN-6971?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16119442#comment-16119442 ]
Sunil G commented on YARN-6971: ------------------------------- Post YARN-6775, we use SimpleResource for Resource.newInstance(). So for all internal operations on Resource object, we could either use {{Resource.newInstance()}} or {{Resources. createResource()}}. YARN-3926 work is streamlining this in a cleaner way. On that note, we need to use {{BuilderUtils.newResource()}} only in the case where we need to send the resource over rpc (communication between clients and servers) Could you please share more details if I missed some points here. > Clean up different ways to create resources > ------------------------------------------- > > Key: YARN-6971 > URL: https://issues.apache.org/jira/browse/YARN-6971 > Project: Hadoop YARN > Issue Type: Improvement > Components: resourcemanager, scheduler > Reporter: Yufei Gu > Priority: Minor > Labels: newbie > > There are several ways to create a {{resource}} object, e.g., > BuilderUtils.newResource() and Resources.createResource(). These methods not > only cause confusing but also performance issues, for example > BuilderUtils.newResource() is significant slow than > Resources.createResource(). > We could merge them some how, and replace most BuilderUtils.newResource() > with Resources.createResource(). -- 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