[GitHub] nifi issue #2502: NIFI-4165: Added RemoveFlowFilesWithMissingContent.java an...

2018-05-18 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2502 @markap14 sorry I got distracted from this review. I have revisited it and I have some points I'd like to discuss: * I rebased against `master`, as there have obviously been some changes

[GitHub] nifi issue #2502: NIFI-4165: Added RemoveFlowFilesWithMissingContent.java an...

2018-03-02 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/2502 @alopresto This is not really for a corrupted flowfile repository but rather for a flowfile repo that points to content that no longer exists. So the easiest thing would be to create a

[GitHub] nifi issue #2502: NIFI-4165: Added RemoveFlowFilesWithMissingContent.java an...

2018-03-01 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2502 @markap14 do you have an example corrupted flowfile repository to test this against? ---

[GitHub] nifi issue #2502: NIFI-4165: Added RemoveFlowFilesWithMissingContent.java an...

2018-03-01 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2502 Reviewing... ---