ChrisSamo632 commented on PR #7995: URL: https://github.com/apache/nifi/pull/7995#issuecomment-1801443699
@exceptionfactory thanks for the update, the code coverage will help improve confidence in the code base going forward. I've suggeted a couple of options to think about for improving the use of JaCoCo - more for consideration than blockers to merging this PR. As discussed elsewhere, would it be sensible to update the `integration-tests-ci` profile to skip all `ITRedisDistributedMapCacheClientService` ITs as part of this PR - more of them failed the first time in the PR checks, but then succeeded on the retry? -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org