Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prune resolved promises (recipient side) #799

Closed
wants to merge 5 commits into from

Conversation

FUDCo
Copy link
Contributor

@FUDCo FUDCo commented Mar 27, 2020

This should cover the recipient side of resolved promises pruning, as described in issue #766.

@FUDCo FUDCo added SwingSet package: SwingSet swingset-runner package: swingset-runner performance Performance related issues labels Mar 27, 2020
@FUDCo FUDCo requested a review from warner March 27, 2020 21:55
@warner
Copy link
Member

warner commented Apr 9, 2020

I think the next step here is to rebase this on top of current trunk (which should fail CI because of the new tests I landed today), then update those tests to match the new behavior. Please re-ping me for review once that's done and CI is passing again.

Copy link
Member

@warner warner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(see previous comment)

@FUDCo
Copy link
Contributor Author

FUDCo commented Apr 13, 2020

Getting rid of this PR in preparation for a cleaner packaging of these changes.

@FUDCo FUDCo closed this Apr 13, 2020
@FUDCo FUDCo mentioned this pull request May 21, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance related issues SwingSet package: SwingSet swingset-runner package: swingset-runner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants