Skip to content

Poling translations infinite loop fix - #200

Merged
foteinigk merged 1 commit into
develfrom
TX-14801_pull_requests_loop
Jul 31, 2023
Merged

Poling translations infinite loop fix#200
foteinigk merged 1 commit into
develfrom
TX-14801_pull_requests_loop

Conversation

@foteinigk

@foteinigk foteinigk commented Jul 31, 2023

Copy link
Copy Markdown
Contributor

If downloading translations fails due to reasons such as the inability to create the downloaded file the "redirect" field in the response is empty, an infinite loop can occur. To avoid this we can break the loop if task's status is failed

If downloading translations fails due
to reasons such as the inability to
create the downloaded file
the  "redirect" field in the
response is empty and status is falied
Based on backoff functionality, a request will be retried after
1s, 1s, 1s, 2s, 3s, 5s, 8s, 13s and then after 13s forever
To avoid this we can break the loop if status of task is failed

Remove retries
@foteinigk
foteinigk force-pushed the TX-14801_pull_requests_loop branch from fd077f8 to 2e68793 Compare July 31, 2023 10:58

@manos-transifex manos-transifex left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nice one 👍

@foteinigk
foteinigk merged commit ee2462a into devel Jul 31, 2023
@foteinigk
foteinigk deleted the TX-14801_pull_requests_loop branch July 31, 2023 11:07
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.

2 participants