Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Write a warning when the AdaptiveImageServlet is not using an already eixsting renditions #2490

Open
joerghoh opened this issue Apr 26, 2023 · 0 comments · May be fixed by #2492
Open

Write a warning when the AdaptiveImageServlet is not using an already eixsting renditions #2490

joerghoh opened this issue Apr 26, 2023 · 0 comments · May be fixed by #2492
Assignees

Comments

@joerghoh
Copy link
Collaborator

joerghoh commented Apr 26, 2023

Feature Request

The creation of custom renditions on-the-fly can be a major problem, especially when it is operating on large asserts. It can lead to high memory consumption, leading to massive garbage collection and even to service outages, when the JVM is throwing an OutOfMemory exception.

For that reason it is recommended to have all required dimensions as renditions (using processing profiles). In that case the AdaptiveImageServlet does not need to resize the image, but just delivers an existing rendition.

To identify such cases there should be a WARN message whenever the AdaptiveImageServlet resizes an image on-the-fly, it should be similar to this one: Adobe-Consulting-Services/acs-aem-commons#3097

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

Successfully merging a pull request may close this issue.

3 participants