steveloughran opened a new pull request, #7048:
URL: https://github.com/apache/hadoop/pull/7048

   
   Disables all logging by the AWS SDK Transfer Manager.
   
   This is done in ClientManagerImpl construction so is automatically done 
during S3AFS initialization.
   
   ITests verify that
   * It is possible to restore the warning log. This verifies the validity of 
the test suite, and will identify when an SDK update fixes this regression.
   * Constructing an S3A FS instance will disable the logging.
   
   The log manipulation code is lifted from Cloudstore, where it was used to 
dynamically enable logging. It uses reflection to load the Log4J binding; all 
uses of the API catch and swallow exceptions.
   This is needed to avoid failures when running against different log backends
   
   This is an emergency fix -we could come up with a better design for the 
reflection based code using the new DynMethods classes. But this is based on 
working code, which is always good.
   
   
   ### How was this patch tested?
   
   
   New ITests
   
   ### For code changes:
   
   - [X] Does the title or this PR starts with the corresponding JIRA issue id 
(e.g. 'HADOOP-17799. Your PR title ...')?
   - [ ] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   
   


-- 
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: [email protected]

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