Skip to content

Commit

Permalink
Limit maven memory on CI
Browse files Browse the repository at this point in the history
Maven process spawns sub processes, including tests processes, so it's
important that the maven process uses limited memory, as the whole
node's memory isn't unbounded.

This restores maven memory settings as of before commit
f133a42.
  • Loading branch information
findepi authored and wendigo committed Mar 26, 2024
1 parent 9e29231 commit 0714cae
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ env:
CONTINUOUS_INTEGRATION: true
# allow overriding Maven command
MAVEN: ./mvnw
MAVEN_OPTS: ""
MAVEN_INSTALL_OPTS: ""
MAVEN_OPTS: "-Xmx512M -XX:+ExitOnOutOfMemoryError"
MAVEN_INSTALL_OPTS: "-Xmx3G -XX:+ExitOnOutOfMemoryError"
MAVEN_FAST_INSTALL: "-B -V --quiet -T 1C -DskipTests -Dmaven.source.skip=true -Dair.check.skip-all"
MAVEN_COMPILE_COMMITS: "-B --quiet -T 1C -DskipTests -Dmaven.source.skip=true -Dair.check.skip-all=true -Dmaven.javadoc.skip=true --no-snapshot-updates --no-transfer-progress -pl '!:trino-server-rpm'"
MAVEN_GIB: "-P gib -Dgib.referenceBranch=refs/remotes/origin/${{ github.event_name == 'pull_request' && github.event.pull_request.base.ref || github.event.repository.default_branch }}"
Expand Down

0 comments on commit 0714cae

Please sign in to comment.