joewitt commented on PR #8536: URL: https://github.com/apache/nifi/pull/8536#issuecomment-2012478802
@slambrose Yes. https://github.com/apache/nifi/actions. When I build locally they work fine and that link suggests they succeed quite often including recently. They still aren't 100% bulletproof and as such do have failures at times and tremendous effort has gone into improving them. But I interpret David's note to mean he looked at them and found it is likely related to this change. Fundamentally if your question "Are these system/integration tests important to verify for changes like this?" The answer is yes they're very important as a safeguard to ensure changes at the framework level have some sanity checking. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
