Status Update
Comments
xa...@google.com <xa...@google.com> #2
Android build
Which Android build are you using? (e.g. UQ1A.240205.002)
Device used
Which device did you use to reproduce this issue?
Are you reporting this as a Developer or User?
Android bug report (to be captured after reproducing the issue)
For steps to capture a bug report, please refer:
Alternate method
Navigate to “Developer options”, ensure “USB debugging” is enabled, then enable “Bug report shortcut”. Capture bug report by holding the power button and selecting the “Take bug report” option.
Note: Please upload the bug report to google drive and share the folder to android-bugreport@google.com, then share the link here.
xa...@google.com <xa...@google.com> #3
Please provide the requested information to proceed further. Unfortunately the issue will be closed within 7 days if there is no further update.
au...@google.com <au...@google.com> #4
We are closing this issue since we didn't receive a response. If you are still facing this problem, please open a new issue and add the relevant information along with reference to this issue.
je...@google.com <je...@google.com> #6
Bingran, can you have a look at this asap. thx !
bi...@google.com <bi...@google.com>
bi...@google.com <bi...@google.com> #7
To make sure I analyze the "buildOnServer-7.5-rc-2-2022-07-01-shutdown.snapshot" correctly, I attached a screenshot of how I find the concerning point. From the picture, the MergeGeneratedProguardFilesCreationAction takes 1924ms. Is that the right way to check your snapshot file? If so, what is the expected time for that creation action in a config cached build?
au...@google.com <au...@google.com> #8
I think you missed adding an attachment.
bi...@google.com <bi...@google.com> #9
oops, here it is =)
au...@google.com <au...@google.com> #10
From discussions with xav@ that time should almost be non-existent during the run when the configuration cache kicks in.
bi...@google.com <bi...@google.com> #11
I checked out androidx-main, is that the right one? There is no gradlew file under that directory.
bingran@bingran-linux-west:~/androidx-main/tools$ ls
external fetch_artifact metalava repohooks
And have you ever tried building androidx with the dev version of AGP(the one built from studio-main)?
au...@google.com <au...@google.com> #12
I checked out androidx-main, is that the right one? There is no gradlew file under that directory.
Yes, aosp androidx-main is the repo. You'll want to use frameworks/support/gradlew
thats the main repo all the work is done.
And have you ever tried building androidx with the dev version of AGP(the one built from studio-main)?
Yes we have a script for that
namely you can set a number of variables to use the latest AGP (
- GRADLE_PLUGIN_VERSION
- LINT_VERSION
- GRADLE_PLUGIN_REPO
and then run frameworks/support/gradlew
bi...@google.com <bi...@google.com> #13
I believe this issue can be fixed by Jerome'CL ag/19955912. I got 4 snapshots running androidx to verify the fix.
- the first run without the fix
- the second run(config cached) without the fix
- the first run with the fix
- the second run(config cached) with the fix
From the snapshot of secondWithFix_xxx
, if we do a search for MergeGeneratedProguardFilesCreationAction
in the call tree, nothing would show up. However, if we do the same search for secondWithoutFix_xx
, 3612ms is spent inside MergeGeneratedProguardFilesCreationAction
.
The fix also reduce the time spent in non config cached run if we compare the same thing for firstWithoutFix_xxx
and firstWithFix_xxx
je...@google.com <je...@google.com> #14
Great bug Aurimas, very useful to get such bugs filed.
bi...@google.com <bi...@google.com> #15
Verified again for the latest patch of ag/19955912. The snapshot of the config cached run and the screenshot of the search result of the MergeGeneratedProguardFilesCreationAction
are attached. With the fix, the MergeGeneratedProguardFilesCreationAction
would only take a 8 ms which is close to zero.
Description
DESCRIBE THE ISSUE IN DETAIL:
MergeGeneratedProguardFilesCreationAction configuration is slow even running the second time when configuration caching is on.
STEPS TO REPRODUCE:
./gradlew bOS
./gradlew bOS
the second timeExpected
MergeGeneratedProguardFilesCreationAction is fast
Actual
MergeGeneratedProguardFilesCreationAction is not fast
Attaching yourkit trace