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

Add GuildImpl#invalidate #1227

Closed
wants to merge 1 commit into from
Closed

Conversation

MinnDevelopment
Copy link
Member

@MinnDevelopment MinnDevelopment commented Mar 12, 2020

Pull Request Etiquette

Changes

  • Internal code
  • Library interface (affecting end-user code)
  • Documentation
  • Other: _____

Closes Issue: NaN

Description

This will be very useful for the new auto-sharding.

@MinnDevelopment MinnDevelopment marked this pull request as ready for review April 7, 2020 12:40
@MinnDevelopment MinnDevelopment marked this pull request as draft June 10, 2020 14:18
@DV8FromTheWorld
Copy link
Member

Is there a reason we didn't go ahead with this change?

@MinnDevelopment
Copy link
Member Author

I added this for the purpose of auto-sharding, but I never got to it, so now it's just stale. To merge this we would need to check if any changes need to be addressed since this PR was created.

@DV8FromTheWorld
Copy link
Member

We should consider resurrecting this as we just encountered a situation where the cache didn't get cleared because invalidation was forgotten. Having this all in one place would be nice. #2029

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

Successfully merging this pull request may close these issues.

None yet

2 participants