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

[20.01] Run email post job action only once per implicit collection jobs #9277

Draft
wants to merge 3 commits into
base: release_20.01
Choose a base branch
from

Conversation

mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Jan 29, 2020

Fixes the remainder of #9275

@mvdbeek mvdbeek changed the title Run email post job action only once per implicit collection jobs [20.01] Run email post job action only once per implicit collection jobs Jan 29, 2020
@mvdbeek mvdbeek added this to the 20.01 milestone Jan 29, 2020
@mvdbeek mvdbeek requested a review from dannon January 29, 2020 12:04
@mvdbeek
Copy link
Member Author

mvdbeek commented Jan 29, 2020

@jmchilton mentioned that this could have a pretty bad performance penalty (each job checking each jobs status for implicit collections). So maybe we shouldn't do this for now / until we have a more performant way to do this.

@mvdbeek mvdbeek removed this from the 20.01 milestone Jan 29, 2020
@nsoranzo nsoranzo marked this pull request as draft September 16, 2022 00:16
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.

3 participants