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

[Text Reading] Set _JAVA_OPTIONS to min. RAM requirements #507

Merged
merged 5 commits into from
Sep 28, 2023

Conversation

myedibleenso
Copy link
Collaborator

Summary of changes

Sets _JAVA_OPTIONS env variable to a default value that illustrates the minimum RAM requirements for the app.

This can be overwritten using the -e flag when launching a container using our TR image.

@myedibleenso myedibleenso added documentation Improvements or additions to documentation Integration Text Reading labels Sep 26, 2023
@myedibleenso myedibleenso added this to the [DARPA] Milestone 10 milestone Sep 26, 2023
@github-actions github-actions bot removed documentation Improvements or additions to documentation Text Reading labels Sep 26, 2023
@myedibleenso myedibleenso requested review from kwalcock and removed request for enoriega September 27, 2023 21:07
Copy link
Contributor

@kwalcock kwalcock left a comment

Choose a reason for hiding this comment

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

LGTM. Hopefully the file.encoding is not necessary.

@myedibleenso myedibleenso merged commit 1c217bb into main Sep 28, 2023
6 checks passed
@myedibleenso myedibleenso deleted the ghp/java-opts-docker branch September 28, 2023 05:44
github-actions bot added a commit that referenced this pull request Sep 28, 2023
## Summary of changes
Sets `_JAVA_OPTIONS` env variable to a default value that illustrates
the minimum RAM requirements for the app.

This can be overwritten using the `-e` flag when launching a container
using our TR image. 1c217bb
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.

3 participants