Skip to content

Conversation

@Firtzberg
Copy link

@Firtzberg Firtzberg commented Sep 21, 2019

Reduces memory usage in (my) tests by several percent.

@GrahamCampbell GrahamCampbell changed the title Flushing [7.x] Improves flushing of application/container Sep 21, 2019
@driesvints
Copy link
Member

@Firtzberg can you provide some benchmarks for this?

@taylorotwell
Copy link
Member

Performance PRs aren't accepted without benchmark comparisons.

@Firtzberg
Copy link
Author

@taylorotwell Consider it code cleanup then.
There are no details on how to make benchmark comparisons in the contributing guide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants