Skip to content

Close the session for preview generation#11352

Merged
LukasReschke merged 1 commit intomasterfrom
close-session-for-previews
Sep 29, 2014
Merged

Close the session for preview generation#11352
LukasReschke merged 1 commit intomasterfrom
close-session-for-previews

Conversation

@LukasReschke
Copy link
Copy Markdown
Member

Without closing the session every preview image generation is locking the session which makes the webinterface unresponsive.

Without closing the session every preview image generation is locking the session which makes the webinterface unresponsive.
@DeepDiver1975
Copy link
Copy Markdown
Member

nice catch 👍

d35ad6eaf09d47cf8b9fc28ba6dfee46d25a752a9253b3b2c4609e4b1c37cd1d

@LukasReschke
Copy link
Copy Markdown
Member Author

@karlitschek We should backport this... I think...

@scrutinizer-notifier
Copy link
Copy Markdown

The inspection completed: No new issues

@MorrisJobke
Copy link
Copy Markdown
Contributor

👍

@karlitschek
Copy link
Copy Markdown
Contributor

indeed very cool. 👍 Please backport. This will make a lot of users happy who complained about the preview performance. :-)

@ghost
Copy link
Copy Markdown

ghost commented Sep 29, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser/7618/

LukasReschke added a commit that referenced this pull request Sep 29, 2014
Close the session for preview generation
@LukasReschke LukasReschke merged commit 35465f5 into master Sep 29, 2014
@LukasReschke LukasReschke deleted the close-session-for-previews branch September 29, 2014 20:03
@LukasReschke
Copy link
Copy Markdown
Member Author

stable7: 7eb0d68

@lock lock bot locked as resolved and limited conversation to collaborators Aug 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants