Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1954

WIP: CMFActivity: remove 1k validation/queue/s rate limit

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Vincent Pelletier requested to merge vpelletier/erp5:cmfactivity_remove_1k_valdiation_per_second_per_queue_rate_limit into master Jun 10, 2024
  • Overview 3
  • Commits 3
  • Pipelines 0
  • Changes 3

I discovered that CMFActivity's validation willingly hands control back to timerserver after a fixed number of activities were successfully validated (1000).

This means it is not possible to validate more than 1k activities per queue per second.

In this merge request, I introduce two preparatory changes and then the main logic change: let CMFActivity.distribute whether Activities.SQLBase.distribute reached the end of the activities it could try to validate (by returning on an empty select result), and only then hand control over to timerserver, similarly to what was already done for processing nodes.

/cc @Nicolas

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: cmfactivity_remove_1k_valdiation_per_second_per_queue_rate_limit
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7