dify icon indicating copy to clipboard operation
dify copied to clipboard

Should we add a default value of `MAX_LIFETIME_SECONDS` to the unstructured docker container in docker compose file

Open CXwudi opened this issue 1 year ago • 1 comments

Self Checks

  • [X] I have searched for existing issues search for existing issues, including closed ones.
  • [X] I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
  • [X] [FOR CHINESE USERS] 请务必使用英文提交 Issue,否则会被关闭。谢谢!:)
  • [X] Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing? Tell me about your story.

If you self-host Unstructured, you might experience memory leak issue from https://github.com/Unstructured-IO/unstructured-api/issues/197

Fortunately, based on the dev's reply and the README. You can set MAX_LIFETIME_SECONDS to some value to shut down the server to work around the memory leak issue with restart=always docker flag.

Hence, I am wondering, should we add MAX_LIFETIME_SECONDS to the docker compose file? If not, maybe shall we mention the memory leak issue in a documentation or a comment in the docker compose file to let the user aware?

2. Additional context or comments

No response

3. Can you help us with this feature?

  • [ ] I am interested in contributing to this feature.

CXwudi avatar Aug 26 '24 22:08 CXwudi

Hence, I am wondering, should we add MAX_LIFETIME_SECONDS to the docker compose file? If not, maybe shall we mention the memory leak issue in a documentation or a comment in the docker compose file to let the user aware?

Thanks for the suggestion, I will add it later.

crazywoola avatar Aug 27 '24 01:08 crazywoola