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.

Sign up for GitHub

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

Jump to bottom

Memory usage tweaks to improve usage of memory #3951

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Memory usage tweaks to improve usage of memory #3951

wants to merge 5 commits into from

Conversation

sc0ttkclark
Copy link
Member

@sc0ttkclark sc0ttkclark commented Jan 19, 2017 •
edited
Loading

Tweaks to when to load up objects and avoid getting ALL OBJECTS on load. This avoids filling memory up for larger config sites

@sc0ttkclark sc0ttkclark added Status: Holding Status: In Progress Issue or PR is currently in progress but not yet done Patch labels Jan 19, 2017
@sc0ttkclark sc0ttkclark added this to the Pods Future Release milestone Jan 19, 2017
@sc0ttkclark sc0ttkclark self-assigned this Jan 19, 2017
@sc0ttkclark sc0ttkclark changed the base branch from 2.x to release/2.7 June 1, 2017 19:12
@sc0ttkclark sc0ttkclark requested a review from pglewis June 1, 2017 19:12
@sc0ttkclark
Copy link
Member Author

There may be some issues with this and i18n integration etc with some post type/taxonomy labels, we should test and confirm.

This also needs conflict resolution.

@pglewis
Copy link
Contributor

pglewis commented Jul 27, 2017

Any status change?

@sc0ttkclark sc0ttkclark changed the base branch from release/2.7 to release/2.8 December 6, 2017 06:53
@pglewis pglewis removed the Patch label Jun 12, 2018
@sc0ttkclark sc0ttkclark removed the request for review from pglewis March 9, 2020 05:39
Base automatically changed from release/2.8 to main October 18, 2021 17:07
@JoryHogeveen
Copy link
Member

@sc0ttkclark Was this review request automation?

@sc0ttkclark
Copy link
Member Author

Yes, looks like it saw no reviewers set and it just auto-set you

@sc0ttkclark sc0ttkclark marked this pull request as draft October 18, 2021 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoryHogeveen JoryHogeveen Awaiting requested review from JoryHogeveen JoryHogeveen is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@sc0ttkclark sc0ttkclark

Labels
Status: Holding Status: In Progress Issue or PR is currently in progress but not yet done
Projects
None yet
Milestone
Backlog
Development

Successfully merging this pull request may close these issues.
3 participants
@sc0ttkclark @pglewis @JoryHogeveen