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

ref(debug-files): Buffer updates to reprocessing revision project option #29041

Merged
merged 1 commit into from
Oct 4, 2021

Conversation

beezz
Copy link
Contributor

@beezz beezz commented Oct 4, 2021

No description provided.

@beezz beezz requested review from jan-auer and untitaker October 4, 2021 14:58
ProjectOption,
columns={},
filters={"project": project, "key": REPROCESSING_OPTION},
signal_only=True,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

signal_only=True so the actual update happens in receiver and will properly reload cache via the ProjectOption.objects.set_value.

Copy link
Member

Choose a reason for hiding this comment

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

I absolutely love how you have to specify a model cls that, with signal_only, is only used as input for the redis key and nothing else.

@beezz beezz marked this pull request as ready for review October 4, 2021 15:29
@beezz beezz requested a review from a team October 4, 2021 15:29
Copy link
Member

@untitaker untitaker left a comment

Choose a reason for hiding this comment

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

Yeah that's exactly the unholy implementation I had in mind. I'll let jan confirm whether debouncing this is actually safe to do.

@beezz beezz merged commit 186767c into master Oct 4, 2021
@beezz beezz deleted the ref/buffer-reprocessing-revision-updates branch October 4, 2021 17:59
@github-actions github-actions bot locked and limited conversation to collaborators Oct 20, 2021
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