Skip to content

[14.0] [FIX] base_deterministic_session_gc: out of memory error - #3447

Merged
OCA-git-bot merged 1 commit into
OCA:14.0from
Therp:14.0-base_deterministic_session_gc-memory-efficiency
Dec 5, 2025
Merged

[14.0] [FIX] base_deterministic_session_gc: out of memory error#3447
OCA-git-bot merged 1 commit into
OCA:14.0from
Therp:14.0-base_deterministic_session_gc-memory-efficiency

Conversation

@thomaspaulb

@thomaspaulb thomaspaulb commented Nov 15, 2025

Copy link
Copy Markdown
Contributor

Looping over a large amount of session files with os.listdir caused memory issues. glob.iglob is a generator and as such more memory-efficient.

Analogous to how Odoo 16+ does it

@thomaspaulb thomaspaulb changed the title [FIX] base_deterministic_session_gc: out of memory error [14.0] [FIX] base_deterministic_session_gc: out of memory error Nov 15, 2025
Comment thread base_deterministic_session_gc/http.py Outdated
Looping over a large amount of session files with os.listdir caused memory issues.
glob.iglob is a generator and as such more memory-efficient.
@thomaspaulb
thomaspaulb force-pushed the 14.0-base_deterministic_session_gc-memory-efficiency branch from 6ce0912 to 0f3d351 Compare November 17, 2025 13:02

@rrebollo rrebollo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review. LGTM!

@thomaspaulb

Copy link
Copy Markdown
Contributor Author

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3447-by-thomaspaulb-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 249207e into OCA:14.0 Dec 5, 2025
8 of 9 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@thomaspaulb
thomaspaulb deleted the 14.0-base_deterministic_session_gc-memory-efficiency branch December 5, 2025 14:51
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.

5 participants