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

ASF GitHub Bot commented on HADOOP-18371:
-----------------------------------------

ayushtkn commented on code in PR #4730:
URL: https://github.com/apache/hadoop/pull/4730#discussion_r944761833


##########
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java:
##########
@@ -1001,7 +1001,7 @@ protected RequestFactory createRequestFactory() {
     try {
       storageClass = StorageClass.fromValue(storageClassConf);
     } catch (IllegalArgumentException e) {
-      LOG.warn("Unknown storage class property {}: {}; falling back to default 
storage class",
+      LOG.info("Unknown storage class property {}: {}; falling back to default 
storage class",

Review Comment:
   If you don't configure anything and then if it is falling back to default, 
it should be info in that case, but if someone configured it and configured it 
wrong it should be warn only right? it isn't the normal flow of execution





> s3a FS init logs at warn if fs.s3a.create.storage.class is unset
> ----------------------------------------------------------------
>
>                 Key: HADOOP-18371
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18371
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.3.9
>            Reporter: Steve Loughran
>            Assignee: Viraj Jasani
>            Priority: Blocker
>              Labels: pull-request-available
>
> if you don't have an s3a storage class set in 
> {{fs.s3a.create.storage.class}}, then whenever you create an S3A FS instance, 
> it logs at warn
> {code}
> bin/hadoop s3guard bucket-info $BUCKET
> 2022-07-27 11:53:11,239 [main] INFO  Configuration.deprecation 
> (Configuration.java:logDeprecation(1459)) - fs.s3a.server-side-encryption.key 
> is deprecated. Instead, use fs.s3a.encryption.key
> 2022-07-27 11:53:11,240 [main] INFO  Configuration.deprecation 
> (Configuration.java:logDeprecation(1459)) - 
> fs.s3a.server-side-encryption-algorithm is deprecated. Instead, use 
> fs.s3a.encryption.algorithm
> 2022-07-27 11:53:11,396 [main] WARN  s3a.S3AFileSystem 
> (S3AFileSystem.java:createRequestFactory(1004)) - Unknown storage class 
> property fs.s3a.create.storage.class: ; falling back to default storage class
> 2022-07-27 11:53:11,839 [main] INFO  impl.DirectoryPolicyImpl 
> (DirectoryPolicyImpl.java:getDirectoryPolicy(189)) - Directory markers will 
> be kept
> Filesystem s3a://stevel-london
> Location: eu-west-2
> {code}
> note, this is why part of quaifying an sdk update involves looking at the 
> logs and running the CLI commands by hand...you see if new messages have 
> crept in



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

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

Reply via email to