[NIFI-5849] ListSFTP, ListFTP processors lose Cluster state on restart.

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.8.0
• Fix Version/s: 1.9.0
• Component/s: Extensions
• Assignee: Koji Kawamura
• Reporter: Truong Duc Kien
• Created: 29/Nov/18 05:52
• Updated: 29/Nov/18 09:32
• Resolved: 29/Nov/18 09:32

Description

In a cluster environment, restarting a processor that extends AbstractListProcessor could lead to the erasure of that processor's existing cluster states.

This bug was introduced in commit https://github.com/apache/nifi/commit/8b9d4461185848fd552a639ac14b7926164d5426 .

Before that commit, the resetState variable is changed to false after the state is clear, inside updateState method. However, the commit moved this line to inside the onTrigger method. This is problematic, because onTrigger method is only called on the primary node. As a result, the resetState variable is only reset to false on primary node, but not on the other nodes. When the user restarts the processor,  every nodes will re-execute the updateState method and the Cluster state will be cleared.

Issue Links

• links to: GitHub Pull Request #3189