YolandaMDavis commented on issue #3874: NIFI-6842 - Introduce MetricsEventReportingTask URL: https://github.com/apache/nifi/pull/3874#issuecomment-551218728 > Looking good, made one doc-related comment. Also this might be a good time to change `test_nifi_rules.yml` and its associated `TestEasyRulesEngineService.testYamlNiFiRules()` test, the rule fires when backpressure is predicted to be more than 5 mins, not less. The condition should probably be `predictedTimeToBytesBackpressureMillis > 0 && predictedTimeToBytesBackpressureMillis < 300000` and the unit test should test for a value in that range. Looks like `TestRulesFactory.confirmEntries()` would have to be updated as well, for the changed condition. Thanks for the feedback @mattyb149. I'm actually editing this test as part of NIFI-6854. If it's ok with you I can include that fix there (should have a PR ready today)?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
