[GitHub] [nifi] Lehel44 commented on a diff in pull request #6047: NIFI-10028: Added QuerySalesforceObject additionalDetails

[GitBox <gi...@apache.org>]

Lehel44 commented on code in PR #6047:
URL: https://github.com/apache/nifi/pull/6047#discussion_r873907665


##########
nifi-nar-bundles/nifi-salesforce-bundle/nifi-salesforce-processors/src/main/java/org/apache/nifi/processors/salesforce/QuerySalesforceObject.java:
##########
@@ -182,9 +182,8 @@ public class QuerySalesforceObject extends AbstractProcessor {
     static final PropertyDescriptor AGE_DELAY = new PropertyDescriptor.Builder()
             .name("age-delay")
             .displayName("Age Delay")
-            .description("When 'Age Field' is set the age-based filter will be adjusted by this amount."
-                    + " Only records that are older than the previous run time of this processor, by at least this amount, will be retrieved."
-            )
+            .description("A time window is always between the previous run time of the processor and the current run time. " +
+                    "When 'Age Field' is set, the time window's right edge will be adjusted left by this amount.")

Review Comment:
   My intention with introducing the time window concept was to refer to the left and right sides. This property has no conditional nature on the time window itself. It just adjusts the end of it when it is configured. Time window filtering applies when the Age Field is set.



-- 
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: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org