don't warn on EnvironmentCredProvider

ahmarsuhail · ahmarsuhail · commit 35c78b998c12 · 2022-10-05T17:38:21.000+01:00
diff --git a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AUtils.java b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AUtils.java
@@ -640,7 +640,8 @@ public static AWSCredentialProviderList buildAWSProviderList(
     AWSCredentialProviderList providers = new AWSCredentialProviderList();
     for (Class<?> aClass : awsClasses) {
 
-      if (aClass.getName().contains(AWS_AUTH_CLASS_PREFIX)) {
+      if (!aClass.getSimpleName().equals("EnvironmentVariableCredentialsProvider")
+          && aClass.getName().contains(AWS_AUTH_CLASS_PREFIX)) {
         V2Migration.v1ProviderReferenced(aClass.getName());
       }
 

Original file line number	Diff line number	Diff line change
`@@ -640,7 +640,8 @@ public static AWSCredentialProviderList buildAWSProviderList(`
`640`	`640`	`AWSCredentialProviderList providers = new AWSCredentialProviderList();`
`641`	`641`	`for (Class<?> aClass : awsClasses) {`
`642`	`642`
`643`		`- if (aClass.getName().contains(AWS_AUTH_CLASS_PREFIX)) {`
	`643`	`+ if (!aClass.getSimpleName().equals("EnvironmentVariableCredentialsProvider")`
	`644`	`+ && aClass.getName().contains(AWS_AUTH_CLASS_PREFIX)) {`
`644`	`645`	`V2Migration.v1ProviderReferenced(aClass.getName());`
`645`	`646`	`}`
`646`	`647`