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 there. These fall into a couple places: ** the version bump to `1.7.0-SNAPSHOT` in the `pom.xml` for both this artifact and a dependency ** there have been changes to `FlowFileQueue` which `DummyFlowFileQueue` must implement * I added some logic to `RemoveFlowFilesWithMissingContent` which loads the *master key* from the expected `bootstrap.conf` file in order to handle a `nifi.properties` file with encrypted configuration values. * The other NiFi Toolkit components have a `*.bat`/`*.sh` script which allows them to be run. This provides a couple features: ** named command-line arguments as opposed to positional arguments ** Setting up `$JAVA_HOME` and the classpath rather than calling `java` directly on the command-line * The `jar-with-dependencies` in `maven-assembly-plugin` only seems to run when you use `mvn clean compile assembly:single` rather than being tied to the `install` phase via a profile (see [Stack Overflow](https://stackoverflow.com/a/574650/70465)). Please let me know if I'm missing something here I ran the scenario you suggested by generating some flowfiles into a queue and then removing the `content_repository` directory contents. When I did that, I got this message: ``` hw12203:/Users/alopresto/Workspace/nifi/nifi-toolkit/nifi-toolkit-flowfile-repo (pr2502) alopresto ð 149s @ 17:17:25 $ cd target/ hw12203:...ers/alopresto/Workspace/nifi/nifi-toolkit/nifi-toolkit-flowfile-repo/target (pr2502) alopresto ð 0s @ 17:17:31 $ java -cp nifi-toolkit-flowfile-repo-1.7.0-SNAPSHOT-jar-with-dependencies.jar:../../nifi-toolkit-assembly/target/nifi-toolkit-1.7.0-SNAPSHOT-bin/nifi-toolkit-1.7.0-SNAPSHOT/lib/slf4j-api-1.7.25.jar org.apache.nifi.toolkit.repos.flowfile.RemoveFlowFilesWithMissingContent ~/Workspace/nifi/nifi-assembly/target/nifi-1.7.0-SNAPSHOT-bin/nifi-1.7.0-SNAPSHOT/conf/nifi.properties ~/Workspace/nifi/nifi-assembly/target/nifi-1.7.0-SNAPSHOT-bin/nifi-1.7.0-SNAPSHOT/flowfile_repository/ 17:17:35.865 [main] INFO org.apache.nifi.properties.NiFiPropertiesLoader - Loaded 148 properties from /Users/alopresto/Workspace/nifi/nifi-assembly/target/nifi-1.7.0-SNAPSHOT-bin/nifi-1.7.0-SNAPSHOT/conf/nifi.properties 17:17:35.872 [main] DEBUG org.apache.nifi.properties.ProtectedNiFiProperties - Loaded 148 properties (including 0 protection schemes) into ProtectedNiFiProperties 17:17:35.872 [main] DEBUG org.apache.nifi.properties.ProtectedNiFiProperties - No protected properties Cannot find or cannot read ./content_repository or it is not a directory hw12203:...ers/alopresto/Workspace/nifi/nifi-toolkit/nifi-toolkit-flowfile-repo/target (pr2502) alopresto ð 0s @ 17:17:36 $ ``` The directory definitely exists: ``` hw12203:...space/nifi/nifi-assembly/target/nifi-1.7.0-SNAPSHOT-bin/nifi-1.7.0-SNAPSHOT (pr2502) alopresto ð 0s @ 17:17:48 $ ll total 416 drwxr-xr-x 17 alopresto staff 578B May 10 16:40 ./ drwxr-xr-x 3 alopresto staff 102B May 10 10:20 ../ -rw-r--r-- 1 alopresto staff 119K Mar 13 17:25 LICENSE -rw-r--r-- 1 alopresto staff 80K May 10 09:23 NOTICE -rw-r--r-- 1 alopresto staff 4.4K Dec 13 15:56 README drwxr-xr-x 8 alopresto staff 272B May 10 10:20 bin/ drwxr-xr-x 12 alopresto staff 408B May 18 16:51 conf/ drwxr-xr-x 2 alopresto staff 68B May 18 16:51 content_repository/ drwxr-xr-x 6 alopresto staff 204B May 18 16:50 database_repository/ drwxr-xr-x 3 alopresto staff 102B May 10 10:20 docs/ drwxr-xr-x 5 alopresto staff 170B May 18 16:52 flowfile_repository/ drwxr-xr-x 113 alopresto staff 3.8K May 10 10:20 lib/ drwxr-xr-x 10 alopresto staff 340B May 18 17:00 logs/ drwxr-xr-x 9 alopresto staff 306B May 18 16:51 provenance_repository/ drwxr-xr-x 4 alopresto staff 136B May 18 16:49 run/ drwxr-xr-x 3 alopresto staff 102B May 10 16:40 state/ drwxr-xr-x 5 alopresto staff 170B May 18 16:50 work/ hw12203:...space/nifi/nifi-assembly/target/nifi-1.7.0-SNAPSHOT-bin/nifi-1.7.0-SNAPSHOT (pr2502) alopresto ð 0s @ 17:19:35 $ ll content_repository/ total 0 drwxr-xr-x 2 alopresto staff 68B May 18 16:51 ./ drwxr-xr-x 17 alopresto staff 578B May 10 16:40 ../ ``` I believe this is because in the default `nifi.properties` file, the content repository is defined as a relative path `./content_repository`, so I think there should be code in the tool to resolve this path if it is not absolute. Let me know what you think about those comments. I pushed my changes [to a branch NIFI-4165](https://github.com/alopresto/nifi/commit/579da9117d03ad9cc24499bbad6d27fae7c92037).
---