jthurne commented on pull request #10490: URL: https://github.com/apache/kafka/pull/10490#issuecomment-814912578
> Thanks for the PR! Looks like the build failed: Whoops! That was silly of me. Somehow I accidentally deleted an important equals sign when getting these commits ready for submission. Should be fixed now (tested locally this time). > Also, have you measured a performance improvement without Gradle Enterprise? For locally executed builds, the build caching can really help avoid re-running tasks with inputs that haven't changed since the last time they were executed. This can save a lot of time when working locally. You'll observe a noticeable performance gain when running clean builds. Additionally, tests aren't executed anymore if all you do is make a commit that changes something not directly related to the tests themselves, like build logic (although if you change the test tasks, then they'll of course re-run). On the CI-server side, these changes have little to no impact at all, unfortunately. This is already a pretty mature build and the optimizations in this PR are relatively minor. By far, the vast majority of the build time is taken up executing tests. In particular, the tests in the `core` module tend to take a long time to execute, even with parallel test execution enabled. -- 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]
