Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2397#discussion_r161132714
--- Diff:
nifi-nar-bundles/nifi-extension-utils/nifi-reporting-utils/src/main/java/org/apache/nifi/reporting/util/provenance/ProvenanceEventConsumer.java
---
@@ -256,9 +297,15 @@ private boolean isFilteringEnabled() {
}
}
}
+ if (!eventTypesExclude.isEmpty() &&
eventTypesExclude.contains(provenanceEventRecord.getEventType())) {
+ continue;
--- End diff --
These two, `eventTypesExclude` and `eventTypes` are the most
computationally cheap conditions. So, these should be done at the beginning,
before checking ProcessGroup hierarchies. How do you think?
---
