Skip to content
Snippets Groups Projects

Handle concurrency while initializing activity

Merged Valentin Rigal requested to merge activities-concurrency into master
All threads resolved!

Based on backend!1353
Closes #68 (closed)
This would allow to run processes on a similar worker version and element without processing the element twice (that can cause side effects or other errors).
The backend can return a 409_CONFLICT.

  • Add tests

Before: Screenshot_from_2021-05-18_16-35-03 After: Screenshot_from_2021-05-18_16-34-32

Edited by Valentin Rigal

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Valentin Rigal added 1 commit

    added 1 commit

    Compare with previous version

  • Valentin Rigal resolved all threads

    resolved all threads

  • Valentin Rigal added 10 commits

    added 10 commits

    • 7e22020e - 1 commit from branch master
    • 28041ed0 - Handle concurrency while initializing activity
    • d69940d2 - Handle backend 409 conflict error code
    • 00e45a11 - Avoid updating activity to error when we could not fetch the element
    • bd5d15ae - Update existing tests
    • 1eb191bf - Add tests related to activities update
    • 3de131d9 - Suggestions
    • 25d4fff2 - Use a custom conflict Exception
    • 528bc322 - Use the generic error catch and handle exceptions when patching to the Error state
    • 075ebabe - Update tests

    Compare with previous version

  • Valentin Rigal resolved all threads

    resolved all threads

  • Valentin Rigal resolved all threads

    resolved all threads

  • Valentin Rigal added 21 commits

    added 21 commits

    • 075ebabe...6b7ba8d8 - 12 commits from branch master
    • 9a233d00 - Handle concurrency while initializing activity
    • 74e2c4c4 - Handle backend 409 conflict error code
    • 0b7bf018 - Avoid updating activity to error when we could not fetch the element
    • a5dd540b - Update existing tests
    • b3c8d22b - Add tests related to activities update
    • a73202eb - Suggestions
    • cbde9d25 - Use a custom conflict Exception
    • c8326a0e - Use the generic error catch and handle exceptions when patching to the Error state
    • 24766547 - Update tests

    Compare with previous version

  • mentioned in issue #78 (closed)

  • Valentin Rigal added 3 commits

    added 3 commits

    Compare with previous version

  • Valentin Rigal resolved all threads

    resolved all threads

  • Valentin Rigal added 2 commits

    added 2 commits

    Compare with previous version

  • Valentin Rigal added 1 commit

    added 1 commit

    • 42c66c66 - Simplify behavior to avoid confusion

    Compare with previous version

  • Tested locally last version, works :100:

  • Please register or sign in to reply
    Loading