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?
---