Skip to content

Check that a subtask has not already completed before running. - #1388

Merged
brianhw merged 1 commit into
rc/2013-10-17from
brian/fix-email-subtask-dupes
Oct 17, 2013
Merged

Check that a subtask has not already completed before running.#1388
brianhw merged 1 commit into
rc/2013-10-17from
brian/fix-email-subtask-dupes

Conversation

@brianhw

@brianhw brianhw commented Oct 17, 2013

Copy link
Copy Markdown
Contributor

@sarina @rocha Fix for today's release.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What if the task is in progress, not completed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's a problem. This PR doesn't address that. I would need to put in memcache-based locking to address that, and that's more invasive than I'm willing to do right now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That makes sense - however I am a bit concerned that this still won't address the problems we're seeing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It is part of the address. It deals with the situation where there is no overlap. The memcache-based locking does not address that condition. I don't have time right now to get that into this release. Hopefully going forward turning off the heartbeat as Carlos suggests will also help, but when we saw this the other day, the tasks were not overlapping.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK makese sense.

@sarina

sarina commented Oct 17, 2013

Copy link
Copy Markdown
Contributor

👍

brianhw added a commit that referenced this pull request Oct 17, 2013
Check that a subtask has not already completed before running.
@brianhw
brianhw merged commit ff7d906 into rc/2013-10-17 Oct 17, 2013
@rocha

rocha commented Oct 17, 2013

Copy link
Copy Markdown
Contributor

You guys are too fast for me 😦

@sarina

sarina commented Oct 17, 2013

Copy link
Copy Markdown
Contributor

omg carlos your 😦 is just too cute!!!

I will award you a 🐤 and a 🐧

@ichuang ichuang mentioned this pull request Nov 15, 2013
@jzoldak
jzoldak deleted the brian/fix-email-subtask-dupes branch May 5, 2014 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants