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

KTOR-7825 Decrease Gradle memory consumption #4483

Merged
merged 2 commits into from
Nov 26, 2024
Merged

Conversation

osipxd
Copy link
Member

@osipxd osipxd commented Nov 15, 2024

Subsystem
Build Infrastructure

Motivation
It is hard to build the project on machines with less than 16 GB RAM because of high memory consumption.
We have problems on CI when Kotlin Daemon or Gradle Daemon are killed because of high memory pressure.
KTOR-7825 Reduce Gradle memory consumption

Solution

It would be great to specify -XX:MaxMetaspaceSize to prevent Gradle from being silently killed by OS due to unlimited metaspace consumption (see gradle/gradle#19750) and make memory consumption more predictable. Unfortunately, we can't do it now because of Kotlin/binary-compatibility-validator#282

This is a WIP PR for those who are interested in trying the new daemon configurations.

Relates to:

@osipxd osipxd self-assigned this Nov 15, 2024
@osipxd osipxd changed the title Osipxd/decrease memory usage Decrease memory usage Nov 15, 2024
@osipxd osipxd changed the title Decrease memory usage Decrease Gradle memory usage Nov 15, 2024
gradle.properties Outdated Show resolved Hide resolved
build.gradle.kts Outdated Show resolved Hide resolved
@osipxd osipxd changed the title Decrease Gradle memory usage Decrease Gradle memory consumption Nov 15, 2024
@osipxd osipxd force-pushed the osipxd/decrease-memory-usage branch 3 times, most recently from f633a1a to 337ff73 Compare November 21, 2024 15:47
@osipxd osipxd changed the title Decrease Gradle memory consumption KTOR-7825 Decrease Gradle memory consumption Nov 21, 2024
@osipxd osipxd marked this pull request as ready for review November 21, 2024 16:06
@osipxd osipxd force-pushed the osipxd/decrease-memory-usage branch 3 times, most recently from 3fc9ddc to c442534 Compare November 25, 2024 15:12
@osipxd osipxd requested review from e5l and bjhham and removed request for e5l November 26, 2024 11:58
@osipxd osipxd force-pushed the osipxd/decrease-memory-usage branch from c442534 to 29fd734 Compare November 26, 2024 14:14
@osipxd osipxd removed the request for review from bjhham November 26, 2024 14:15
@osipxd osipxd enabled auto-merge (squash) November 26, 2024 14:39
@osipxd osipxd merged commit a136a75 into main Nov 26, 2024
14 checks passed
@osipxd osipxd deleted the osipxd/decrease-memory-usage branch November 26, 2024 16:35
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.

2 participants