[ https://issues.apache.org/jira/browse/YARN-10426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17192874#comment-17192874 ]
Peter Bacsko commented on YARN-10426: ------------------------------------- Small change: TestCSMappingPlacementRule.java: {{import static junit.framework.TestCase.*;}} This should be changed: 1. no star imports 2. {{junit.framework.*}} is an old package, we should import stuff from {{org.junit.Assert.*}} > Cleanup CSMappingPlacementRule remaining minor issues > ----------------------------------------------------- > > Key: YARN-10426 > URL: https://issues.apache.org/jira/browse/YARN-10426 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Gergely Pollak > Assignee: Gergely Pollak > Priority: Minor > > During the reviews of CSMappingPlacementRule changes, there were a few not > urgent changes, which should be done to keep the code as clean as possible, > fix these. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org