-
Notifications
You must be signed in to change notification settings - Fork 4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Automated rollback of commit 14f8b10.
*** Reason for rollback *** b/119318671: ActionRunner holds onto much more memory than FutureTask<ActionExecutionValue>. *** Original change description *** Refactor SkyframeActionExecutor - avoid FutureTask; make ActionRunner self-contained - reorganize the prepare/execute/complete flow This is in preparation for making actions async, which requires that ActionRunner itself becomes async, which in turn precludes the use of FutureTask and also requires that the code flow more cleanly separates pre-execution and post-execution steps. PiperOrigin-RevId: 220866469
- Loading branch information
Showing
3 changed files
with
228 additions
and
189 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.