Github user viazovskyi commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2657#discussion_r189464759
--- Diff:
nifi-nar-bundles/nifi-extension-utils/nifi-processor-utils/src/main/java/org/apache/nifi/processor/util/list/AbstractListProcessor.java
---
@@ -375,7 +376,7 @@ public void onTrigger(final ProcessContext context,
final ProcessSession session
// If our determined timestamp is the same as that
of our last listing, skip this execution as there are no updates
if
(minTimestampToListMillis.equals(this.lastListedLatestEntryTimestampMillis)) {
context.yield();
--- End diff --
I'd like to move the original 'if' with yield out of the loop closer catch,
then I don't need this new 'if' and variable. What do you think?
---
