Skip to content

Implement WorkerActivitiesPerSecond #557

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sokada1221
Copy link
Contributor

@sokada1221 sokada1221 commented Oct 23, 2020

Implements WorkerActivitiesPerSecond as per documentation.
closes #553

Key points:

  • Implements WorkerActivitiesPerSecond with Throttler at PollTaskExecutor
  • Throttler evaluation interval defaults to 1s when interval per message is less than 1ms (i.e. will implicitly allow more bursts)

Test results:

  • Unit tests
  • Local integration tests with workerActivitiesPerSecond set to 0.00001, 0.3, 0.5, 1, 2, 1000, 2000
  • Local integration tests without taskListActivitiesPerSecond set

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WorkerActivitiesPerSecond does not work
1 participant