pyzeebe icon indicating copy to clipboard operation
pyzeebe copied to clipboard

add timeout to ActivateJobs request

Open cx-eyal-goren opened this issue 3 years ago • 13 comments
trafficstars

add timeout to ActivateJobs request handle ZeebeDeadlineExceeded exception

Changes

  • timeout to ActivateJobs request
  • handle ZeebeDeadlineExceeded exception

API Updates

New Features (required)

None

Deprecations (required)

None

Enhancements (optional)

fixing the issue describe below: when using Camunda cloud zeebe, polling requests for activateJobs can stuck forever, which leads to inability to consume more tasks

Checklist

  • [ ] Unit tests
  • [ ] Documentation

References

Fixes #324

cx-eyal-goren avatar Aug 25 '22 09:08 cx-eyal-goren

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 25 '22 09:08 CLAassistant

Hey @EyalCMX, looks good.

Please rebase from master so that tests can run and we can merge

JonatanMartens avatar Nov 06 '22 20:11 JonatanMartens

Hey @EyalCMX, looks good.

Please rebase from master so that tests can run and we can merge

@JonatanMartens done please approve!

cx-eyal-goren avatar Nov 14 '22 14:11 cx-eyal-goren

Pull Request Test Coverage Report for Build 11057437142

Details

  • 6 of 8 (75.0%) changed or added relevant lines in 4 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.08%) to 96.535%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pyzeebe/grpc_internals/zeebe_adapter_base.py 2 3 66.67%
pyzeebe/worker/job_poller.py 0 1 0.0%
<!-- Total: 6 8
Files with Coverage Reduction New Missed Lines %
pyzeebe/worker/job_poller.py 1 83.33%
<!-- Total: 1
Totals Coverage Status
Change from base Build 11056680193: -0.08%
Covered Lines: 1003
Relevant Lines: 1039

💛 - Coveralls

coveralls avatar Nov 14 '22 20:11 coveralls

@EyalCMX the job for format checking fails. Please format using black: poetry run black ., push the changes and I'll approve

JonatanMartens avatar Nov 14 '22 20:11 JonatanMartens

@EyalCMX the job for format checking fails. Please format using black: poetry run black ., push the changes and I'll approve

@JonatanMartens done please approve!

cx-eyal-goren avatar Nov 15 '22 07:11 cx-eyal-goren

@JonatanMartens can u approve it again please

cx-eyal-goren avatar Jan 12 '23 15:01 cx-eyal-goren

@JonatanMartens can u merge it i dont have permission

cx-eyal-goren avatar Jan 25 '23 13:01 cx-eyal-goren

@EyalCMX @JonatanMartens i'm waiting for this to be merged. Is there a more maintained zeebe python plugin available?

michaelarnauts avatar Jan 15 '24 10:01 michaelarnauts

@michaelarnauts i dont think @JonatanMartens maintains this library... and i can't merge it

cx-eyal-goren avatar Jan 15 '24 11:01 cx-eyal-goren

Hey there @michaelarnauts @EyalCMX -- we're looking for additional maintainers for this project. https://github.com/camunda-community-hub/pyzeebe/issues/381

xomiamoore avatar Jan 22 '24 22:01 xomiamoore

Hey @EyalCMX, I'm very sorry for the delayed review

JonatanMartens avatar Feb 21 '24 20:02 JonatanMartens

@JonatanMartens ?

cx-eyal-goren avatar May 07 '24 13:05 cx-eyal-goren

@EyalCMX , congrats!

cx-lior-solomon avatar Sep 29 '24 05:09 cx-lior-solomon