steveloughran commented on a change in pull request #2473:
URL: https://github.com/apache/hadoop/pull/2473#discussion_r530360751



##########
File path: 
hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/AbstractS3ATestBase.java
##########
@@ -60,6 +60,9 @@ public void setup() throws Exception {
     // filesystems which add default configuration resources to do it before
     // our tests start adding/removing options. See HADOOP-16626.
     FileSystem.getLocal(new Configuration());
+    // instantiate an S3A FS here here to force deprecated key load through the
+    // static initializers. See: HADOOP-17385
+    new S3AFileSystem();

Review comment:
       I was about to say I don't know a way to trigger instantiation of static 
class initializers except by a new() call, but actually we can invoke any 
static method. How about I add a new static `initializeClass()` call and load 
it.
   
   FWIW, what I'd really like is to be able to disable forced reload of 
configs, but I can't see a way to do that which wouldn't run the risk of damage




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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to