Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2429#discussion_r175184013
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/LogarithmicallyPenalizeFlowFileProcessor.java
---
@@ -0,0 +1,122 @@
+package org.apache.nifi.processors.standard;
+
+import java.util.Arrays;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+
+import org.apache.nifi.components.PropertyDescriptor;
+import org.apache.nifi.components.ValidationContext;
+import org.apache.nifi.components.ValidationResult;
+import org.apache.nifi.components.Validator;
+import org.apache.nifi.flowfile.FlowFile;
+import org.apache.nifi.processor.AbstractProcessor;
+import org.apache.nifi.processor.ProcessContext;
+import org.apache.nifi.processor.ProcessSession;
+import org.apache.nifi.processor.Relationship;
+import org.apache.nifi.processor.util.StandardValidators;
+
+public class LogarithmicallyPenalizeFlowFileProcessor extends
AbstractProcessor {
+
+ private static final PropertyDescriptor INITIAL_RECONNECT_TIMEOUT =
new PropertyDescriptor.Builder()
+ .name("initialReconnectTimeout")
+ .displayName("InitialReconnectTimeoutMs")
--- End diff --
We should avoid using specific time units in property names or enforcing
time units. Instead, we should call this "Initial Penalty" and use
StandardValidators.TIME_PERIOD_VALIDATOR with a default value of "50 ms".
---
