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

ASF GitHub Bot commented on MAPREDUCE-7420:
-------------------------------------------

slfan1989 commented on code in PR #7363:
URL: https://github.com/apache/hadoop/pull/7363#discussion_r1947556620


##########
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractFSContractTestBase.java:
##########
@@ -250,6 +250,20 @@ protected Path methodPath() throws IOException {
     return path(methodName.getMethodName());
   }
 
+  /**
+   * Get a path whose name ends with the name of this method.
+   *
+   * This is a temporary requirement for upgrading from JUnit 4 to JUnit 5,
+   * and can be deleted later.
+   *
+   * @param name name of this method.
+   * @return a path implicitly unique amongst all methods in this class
+   * @throws IOException IO problems
+   */
+  protected Path methodPath(String name) throws IOException {

Review Comment:
   Thank you for your comment! I have removed the class in the latest code 
submission. Initially, I planned to add this method to `hadoop-common` to 
upgrade all unit tests in the `hadoop-mapreduce-client-core` module. However, I 
found that the test methods in other modules inherit from the test methods of 
the `hadoop-mapreduce-client-core` module, so I temporarily rolled back some 
changes. Unfortunately, I forgot to remove the class. I will complete the 
dependent test upgrades in `mapreduce-client-core` Part2.





> Upgrade Junit 4 to 5 in hadoop-mapreduce-client-core
> ----------------------------------------------------
>
>                 Key: MAPREDUCE-7420
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7420
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: test
>    Affects Versions: 3.3.4
>            Reporter: Ashutosh Gupta
>            Assignee: Ashutosh Gupta
>            Priority: Major
>              Labels: pull-request-available
>
> Upgrade Junit 4 to 5 in hadoop-mapreduce-client-core



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to