tpalfy commented on a change in pull request #3612: NIFI-6486 Add option to use
built-in retry mechanism in AWS processors
URL: https://github.com/apache/nifi/pull/3612#discussion_r310955761
##########
File path:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-abstract-processors/src/main/java/org/apache/nifi/processors/aws/AbstractAWSProcessor.java
##########
@@ -147,6 +174,66 @@
.addValidator(StandardValidators.URL_VALIDATOR)
.build();
+ public static final PropertyDescriptor AWS_MAX_ERROR_RETRY = new
PropertyDescriptor.Builder()
+ .name("max-error-retry")
+ .displayName("Max Error Retry")
+ .description("Maximum number of retry attempts for failed requests.")
+ .defaultValue("0")
+ .addValidator(StandardValidators.NON_NEGATIVE_INTEGER_VALIDATOR)
+ .build();
+
+ public static final PropertyDescriptor AWS_RETRY_POLICY = new
PropertyDescriptor.Builder()
+ .name("retry-policy")
+ .displayName("Retry Policy")
+ .description("Retry policy that can be configured on a specific service
client using ClientConfiguration.")
+ .allowableValues(new HashSet<>(Arrays.asList(
+ RETRY_POLICY_DEFAULT, RETRY_POLICY_DYNAMODB_DEFAULT,
RETRY_POLICY_DEFAULT)))
Review comment:
RETRY_POLICY_DEFAULT appears twice but RETRY_POLICY_CUSTOM is missing.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
