[ https://issues.apache.org/jira/browse/YARN-8501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16545403#comment-16545403 ]
Eric Yang commented on YARN-8501: --------------------------------- [~snemeth] Sorry, I just noticed a nitpick while committing. Test case filename is usually started with Test prefix to ensure the test reports are aggregated correctly. The supporting class files in src/test path don't have Test prefix. This is Hadoop coding style pattern. Could you rename ApplicationsRequestBuilderTest.java to TestApplicationsRequestBuilder.java? Thanks > Reduce complexity of RMWebServices' getApps method > -------------------------------------------------- > > Key: YARN-8501 > URL: https://issues.apache.org/jira/browse/YARN-8501 > Project: Hadoop YARN > Issue Type: Improvement > Components: restapi > Reporter: Szilard Nemeth > Assignee: Szilard Nemeth > Priority: Major > Attachments: YARN-8501.001.patch, YARN-8501.002.patch, > YARN-8501.003.patch, YARN-8501.004.patch > > -- 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