[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-21 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561251354 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DelegationTokenRenewer.java ## @@ -223,7 +223,7 @@ static

[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-20 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561254216 ## File path: hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java ## @@ -100,16 +97,6 @@ public static void

[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-20 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561252276 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java ## @@ -3391,15 +3391,7 @@ private static void

[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-20 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561252276 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java ## @@ -3391,15 +3391,7 @@ private static void

[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-20 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561251612 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DelegationTokenRenewer.java ## @@ -260,8 +259,7 @@ public void

[GitHub] [hadoop] belugabehr commented on a change in pull request #2633: HADOOP-17482: Remove Commons Logger from FileSystem Class

2021-01-20 Thread GitBox
belugabehr commented on a change in pull request #2633: URL: https://github.com/apache/hadoop/pull/2633#discussion_r561251354 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DelegationTokenRenewer.java ## @@ -223,7 +223,7 @@ static