steveloughran commented on code in PR #4483:
URL: https://github.com/apache/hadoop/pull/4483#discussion_r905947970
##########
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/Constants.java:
##########
@@ -1203,4 +1203,18 @@ private Constants() {
* Default maximum read size in bytes during vectored reads : {@value}.
*/
public static final int DEFAULT_AWS_S3_VECTOR_READS_MAX_MERGED_READ_SIZE =
1253376; //1M
+
+ /**
+ * Flag for immediate failure when observing a {@link
AWSBadRequestException}.
+ * If it's disabled and set to false, the failure is treated as retryable.
+ * Value {@value}.
+ */
+ public static final String FAIL_ON_AWS_BAD_REQUEST =
"fs.s3a.fail.on.aws.bad.request";
Review Comment:
I now think "fs.s3a.retry.on.400.response.enabled" would be better, with
default flipped. docs would say "experimental"
and assuming we do have a custom policy, adjacent
```
fs.s3a.retry.on.400.response.delay // delay between attempts, default "10s"
fs.s3a.retry.on.400.response.attempts // number of attempts, default 6
```
--
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]
