steveloughran commented on a change in pull request #3312:
URL: https://github.com/apache/hadoop/pull/3312#discussion_r697593481
##########
File path:
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java
##########
@@ -925,12 +925,16 @@ protected RequestFactory createRequestFactory() {
// request factory.
initCannedAcls(getConf());
+ // Any encoding type
+ String contentEncoding = getConf().get(CONTENT_ENCODING,
DEFAULT_CONTENT_ENCODING);
Review comment:
any content type is valid, even `application/x-shockwave-flash`. there's
probably some rules but it's probably not trying to validate as it'll only
create maintenance. This option should go into storediag for a bit of extra
details...I'll add it once this patch is in
--
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]
