Skip to content

Added java arg for consuming more RAM.#90

Merged
mdellabitta merged 1 commit intomainfrom
use-ram
May 22, 2025
Merged

Added java arg for consuming more RAM.#90
mdellabitta merged 1 commit intomainfrom
use-ram

Conversation

@mdellabitta
Copy link
Contributor

@mdellabitta mdellabitta commented May 22, 2025

Important

Added -XX:MaxRAMPercentage=85 to Java command in Dockerfile to allow Java to use up to 85% of available RAM.

  • Dockerfile:
    • Added -XX:MaxRAMPercentage=85 to the Java command in CMD to allow Java to use up to 85% of available RAM.

This description was created by Ellipsis for 9d1b89e. You can customize this summary. It will automatically update as commits are pushed.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 9d1b89e in 29 seconds. Click for details.
  • Reviewed 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. Dockerfile:15
  • Draft comment:
    Consider making the MaxRAMPercentage value configurable (e.g., via an ENV variable) instead of hardcoding it.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None

Workflow ID: wflow_hcEwb6vnpILPYy5h

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@mdellabitta mdellabitta merged commit 80c1e0e into main May 22, 2025
2 checks passed
@mdellabitta mdellabitta deleted the use-ram branch May 22, 2025 23:48
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 this pull request may close these issues.

1 participant