You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After upgrading this plugin to version 1.1.6 in our large Gradle project, we noticed that the configuration phase duration went from 1 second to anywhere between 20 and 30 seconds. The issue also happens with version 1.1.7. We're using Gradle 8.4, but I've reproduced the issue with Gradle 8.8.
Looking at the commit history, I suspect it's due to caad92a. If I disable applyMavenExclusions, the performance goes back to normal, but doing that is not desirable for us.
Looking at Gradle's Build Scan of the project, the increased duration comes from "Task graph calculation".
On version 1.1.5 (runs slower with --scan, but in reality it's around 1 second):
On version 1.1.6:
Unfortunately, it's difficult to share reproducible build due the NDA, and if I remove most of the code the overall build duration also decreases. However, I'm willing to help test this in any way I can.
The text was updated successfully, but these errors were encountered:
Thanks for the report. From what you've described, I agree that caad92a is the likely cause. The changes were made to fix #384.
Unfortunately, I'm not sure what can be done here to improve the situation. I believe there are two choices:
Use beforeResolve and risk problems with the partial resolution and full resolution disagreeing that may cause Gradle not to execute tasks that should have run. This also triggers a deprecation warning with Gradle 8.8 and is incompatible with Gradle 9.
Use withDependencies as recommended by the Gradle team and suffer the performance hit that you're seeing
Unless there's a third option (@jvandort I'd really welcome your input here please), I think the second option is the better of the two, certainly in the longer term.
but doing that is not desirable for us.
Why is it undesirable? Does it leave you with too many unwanted dependencies that you'd prefer not to exclude through some other means?
Why is it undesirable? Does it leave you with too many unwanted dependencies that you'd prefer not to exclude through some other means?
It's not desirable per se, just not suitable as a quick fix. The performance impact of this is also a significant chunk of the total build time.
For now, we've downgraded to 1.1.5 while I try to figure out what is currently being excluded and whether we can do it in a different way. It's possible there won't be any issues with disabling the Maven exclusions.
After upgrading this plugin to version
1.1.6
in our large Gradle project, we noticed that the configuration phase duration went from 1 second to anywhere between 20 and 30 seconds. The issue also happens with version1.1.7
. We're using Gradle 8.4, but I've reproduced the issue with Gradle 8.8.Looking at the commit history, I suspect it's due to caad92a. If I disable
applyMavenExclusions
, the performance goes back to normal, but doing that is not desirable for us.Looking at Gradle's Build Scan of the project, the increased duration comes from "Task graph calculation".
On version
![Image](https://private-user-images.githubusercontent.com/1769968/408529949-82436c0b-1c85-4b8e-9d0a-2ccd69431f2d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk5NDMyNjUsIm5iZiI6MTczOTk0Mjk2NSwicGF0aCI6Ii8xNzY5OTY4LzQwODUyOTk0OS04MjQzNmMwYi0xYzg1LTRiOGUtOWQwYS0yY2NkNjk0MzFmMmQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxOSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTlUMDUyOTI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YjEwOTc3NmJkMWQwYjgzOWMxNDAzNjdlZDAzNzQ1Nzc3ZjExODFmOTAzNTk3N2EyNGRjZjY3ZjZmMWZkMDQyOCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.KfQQmdHo6Smuv0oyfYYtE4hWK3iEUMp2iF1OfFMFg04)
1.1.5
(runs slower with--scan
, but in reality it's around 1 second):On version
![Image](https://private-user-images.githubusercontent.com/1769968/408530127-f7ef91d6-853f-40ef-8fb6-aee932a78d6a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk5NDMyNjUsIm5iZiI6MTczOTk0Mjk2NSwicGF0aCI6Ii8xNzY5OTY4LzQwODUzMDEyNy1mN2VmOTFkNi04NTNmLTQwZWYtOGZiNi1hZWU5MzJhNzhkNmEucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxOSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTlUMDUyOTI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZjUxOWUyODg3MjUzNzY1ZDgwN2U0YzZlNWEwZTZkODZlZjA1MjI2NmY5Y2QyMzVhYTk0ZjZkMmIzY2ExYjg2YyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.Xa8FWn43KJ8QXYProgko1_nWZny0JRf7onxhHfaMmtU)
1.1.6
:Unfortunately, it's difficult to share reproducible build due the NDA, and if I remove most of the code the overall build duration also decreases. However, I'm willing to help test this in any way I can.
The text was updated successfully, but these errors were encountered: