Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Allow streaming cache invalidate all to workers. #6749

Merged
merged 5 commits into from
Jan 22, 2020

Conversation

erikjohnston
Copy link
Member

No description provided.

@erikjohnston erikjohnston requested a review from a team January 21, 2020 10:51
Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

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

lgtm, just a q/clarification

"""

if cache_name == CURRENT_STATE_CACHE_NAME and keys is None:
raise Exception(
"Can't stream invalidate all with magic current state cache"
Copy link
Member

Choose a reason for hiding this comment

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

As someone who hasn't completely wrapped his head around our caching yet, what does "magic" mean in this context?

Copy link
Member Author

Choose a reason for hiding this comment

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

@erikjohnston erikjohnston merged commit 5d7a6ad into develop Jan 22, 2020
@erikjohnston erikjohnston deleted the erikj/stream_invalidate_all branch February 5, 2020 17:35
babolivier pushed a commit that referenced this pull request Sep 1, 2021
* commit '5d7a6ad22':
  Allow streaming cache invalidate all to workers. (#6749)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants