[jira] [Comment Edited] (YARN-10426) Cleanup CSMappingPlacementRule remaining minor issues

["Peter Bacsko (Jira)" <ji...@apache.org>]

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

Peter Bacsko edited comment on YARN-10426 at 9/9/20, 1:33 PM:
--------------------------------------------------------------

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.*}}


was (Author: pbacsko):
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-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org