Skip to content
This repository has been archived by the owner on Jan 5, 2019. It is now read-only.

Bug 1486071 - simplify description of intermittent-task #298

Closed
wants to merge 1 commit into from

Conversation

djmitche
Copy link
Contributor

@djmitche djmitche commented Nov 9, 2018

..and replace some vague and confusing admonitions with clearer
language.

@djmitche djmitche requested a review from jhford November 9, 2018 00:29
@djmitche djmitche self-assigned this Nov 9, 2018
docs/worker-interaction.md Outdated Show resolved Hide resolved
docs/worker-interaction.md Outdated Show resolved Hide resolved

Reporting a task _exception_ with reason `intermittent-task` will retry the
Copy link
Contributor

Choose a reason for hiding this comment

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

This should continue to discourage using intermittent-task where possible.

..and replace some vague and confusing admonitions with clearer
language.
Copy link
Contributor

@jhford jhford left a comment

Choose a reason for hiding this comment

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

Thanks for addressing some of the concerns! One of the issues has not been addressed, around discouraging the use of intermittent-task though. Was that intentional?

@djmitche
Copy link
Contributor Author

The second two paragraphs are dedicated to such discouragement. What would you like to see?

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.

None yet

2 participants