Stop caching managedFields and last-applied-configuration #20
Conversation
This will save on memory usage in the cache. Signed-off-by: mprahl <[email protected]>
|
JustinKuli
left a comment
JustinKuli
left a comment
There was a problem hiding this comment.
/hold
Looks good to me, definitely helpful as-is. I only wonder if we want to make this even more configurable, or if anyone would want these fields... But since we're the only ones using it so far, those might be unnecessary feature requests
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JustinKuli, mprahl The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/unhold after discussion |
2 participants
This will save on memory usage in the cache.