[ https://issues.apache.org/jira/browse/YARN-9051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16698889#comment-16698889 ]
Antal Bálint Steinbach commented on YARN-9051: ---------------------------------------------- LGTM (+1 Non-binding) > Integrate multiple CustomResourceTypesConfigurationProvider implementations > into one > ------------------------------------------------------------------------------------ > > Key: YARN-9051 > URL: https://issues.apache.org/jira/browse/YARN-9051 > Project: Hadoop YARN > Issue Type: Improvement > Reporter: Szilard Nemeth > Assignee: Szilard Nemeth > Priority: Minor > Attachments: YARN-9051.001.patch > > > CustomResourceTypesConfigurationProvider (extends LocalConfigurationProvider) > has 5 implementations on trunk nowadays. > These could be integrated into 1 common class. > Also, > {{org.apache.hadoop.yarn.util.resource.TestResourceUtils#addNewTypesToResources}} > has similar functionality so this can be considered as well. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org