Github user mcgilman commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2119#discussion_r137876726
--- Diff:
nifi-nar-bundles/nifi-provenance-repository-bundle/nifi-persistent-provenance-repository/src/main/java/org/apache/nifi/provenance/PersistentProvenanceRepository.java
---
@@ -521,6 +522,23 @@ public RepositoryConfiguration getConfiguration() {
return configuration;
}
+ @Override
+ public Set<String> getContainerNames() {
+ return new
HashSet<>(configuration.getStorageDirectories().keySet());
+ }
+
+ @Override
+ public long getContainerCapacity(final String containerName) throws
IOException {
+ final Path path =
configuration.getStorageDirectories().get(containerName).toPath();
--- End diff --
`FileUtils.getContainerCapacity` verifies the containerName is non null.
However, if the specified container is unknown to this repository it will fail
here prior to invoking the `FileUtils`. This issue exists in a number of places
in this PR.
---
