[PR #8783] [MERGED] Allow limiting parallel image encodings to reduce memory usage #11673

New issue

Closed

opened 2025-12-22 08:53:12 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 08:53:12 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/8783
Author: @hannesbraun
Created: 11/21/2022
Status: ✅ Merged
Merged: 1/16/2023
Merged by: @crobibero

Base: master ← Head: limit-parallel-image-encoding

📝 Commits (1)

c8b4c89 Allow limiting parallel image encodings to reduce memory usage

📊 Changes

2 files changed (+35 additions, -2 deletions)

View changed files

📝 MediaBrowser.Model/Configuration/ServerConfiguration.cs (+6 -0)
📝 src/Jellyfin.Drawing/ImageProcessor.cs (+29 -2)

📄 Description

Changes
This change allows specifying the environment variable JELLYFIN_PARALLEL_IMAGE_ENCODING_LIMIT to limit the number of parallel image encodings as those tend to consume a lot of memory (potentially crashing the process as the system runs out of memory). If set, the number of parallel encodings will be limited to that number. Otherwise, the behavior should be unaffected.

The usage of an environment variable made sense here to me, as this setting conceptually didn't really seem to fit into the regular application settings.

Issues
This solves the issue I described in this comment: https://github.com/jellyfin/jellyfin/issues/6877#issuecomment-1257031371

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/8783 **Author:** [@hannesbraun](https://github.com/hannesbraun) **Created:** 11/21/2022 **Status:** ✅ Merged **Merged:** 1/16/2023 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `limit-parallel-image-encoding` --- ### 📝 Commits (1) - [`c8b4c89`](https://github.com/jellyfin/jellyfin/commit/c8b4c8910535679cd42a9921dafe2c982d2bf049) Allow limiting parallel image encodings to reduce memory usage ### 📊 Changes **2 files changed** (+35 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Model/Configuration/ServerConfiguration.cs` (+6 -0) 📝 `src/Jellyfin.Drawing/ImageProcessor.cs` (+29 -2) </details> ### 📄 Description **Changes** This change allows specifying the environment variable `JELLYFIN_PARALLEL_IMAGE_ENCODING_LIMIT` to limit the number of parallel image encodings as those tend to consume a lot of memory (potentially crashing the process as the system runs out of memory). If set, the number of parallel encodings will be limited to that number. Otherwise, the behavior should be unaffected. The usage of an environment variable made sense here to me, as this setting conceptually didn't really seem to fit into the regular application settings. **Issues** This solves the issue I described in this comment: https://github.com/jellyfin/jellyfin/issues/6877#issuecomment-1257031371 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>