Skip to content

Reland "Enable caching of CPU samples collected at application startup (#89600)"#90611

Merged
bkonyi merged 5 commits intomasterfrom
reland_cpu_sample_caching
Oct 13, 2021
Merged

Reland "Enable caching of CPU samples collected at application startup (#89600)"#90611
bkonyi merged 5 commits intomasterfrom
reland_cpu_sample_caching

Conversation

@bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Sep 23, 2021

This reverts commit 58944e6.

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Sep 23, 2021
@google-cla google-cla bot added the cla: yes label Sep 23, 2021
@bkonyi
Copy link
Contributor Author

bkonyi commented Sep 23, 2021

FYI @chunhtai, this caused a bunch of regressions last time it landed. This should have been fixed in the SDK last week so we're going to attempt to reland, but I wanted to keep the gardener in the loop just in case things blow up again.

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@bkonyi
Copy link
Contributor Author

bkonyi commented Sep 28, 2021

google3 check is failing since the engine revision including the fix hasn't been rolled in yet.

@bkonyi bkonyi force-pushed the reland_cpu_sample_caching branch from c873acc to 1ad4546 Compare October 7, 2021 22:43
@bkonyi
Copy link
Contributor Author

bkonyi commented Oct 13, 2021

Only took 3 weeks, but I'm going to finally land this.

@bkonyi bkonyi merged commit e89a885 into master Oct 13, 2021
@bkonyi bkonyi deleted the reland_cpu_sample_caching branch October 13, 2021 14:13
bkonyi added a commit that referenced this pull request Oct 13, 2021
bkonyi added a commit that referenced this pull request Oct 13, 2021
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Oct 29, 2021
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants