Skip to content

[18.0][IMP] queue_job: use __slots__ for ChannelJob#778

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
florentx:18.0_channeljob_slots
Jun 3, 2025
Merged

[18.0][IMP] queue_job: use __slots__ for ChannelJob#778
OCA-git-bot merged 1 commit into
OCA:18.0from
florentx:18.0_channeljob_slots

Conversation

@florentx

@florentx florentx commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

It decreases memory footprint when there's thousands of jobs to prioritize.

It decreases memory footprint when there's thousands of jobs to prioritize.
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @guewen,
some modules you are maintaining are being modified, check this out!

@amh-mw amh-mw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review only.

@simahawk

simahawk commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-778-by-simahawk-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3bedc12 into OCA:18.0 Jun 3, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 1c074f9. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants