You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/06/14 10:20:05 UTC

[GitHub] [flink] gaborgsomogyi commented on a diff in pull request #19953: [FLINK-28044][runtime][security] Make hadoop filesystems configuration available to all deployment targets

gaborgsomogyi commented on code in PR #19953:
URL: https://github.com/apache/flink/pull/19953#discussion_r896642848


##########
flink-filesystems/flink-hadoop-fs/src/test/java/org/apache/flink/runtime/util/HadoopUtilsTest.java:
##########
@@ -49,7 +49,7 @@ public static void setPropertiesToEnableKerberosConfigInit() throws KrbException
     }
 
     @AfterClass
-    public static void cleanupHadoopConfigs() throws KrbException {

Review Comment:
   This is just a cosmetic change because the function doesn't throw anything.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org