Skip to content

command/run: add default GOMEMLIMIT#630

Merged
mmatczuk merged 1 commit intomainfrom
mmt/auto_gomemlimit
Jan 5, 2024
Merged

command/run: add default GOMEMLIMIT#630
mmatczuk merged 1 commit intomainfrom
mmt/auto_gomemlimit

Conversation

@mmatczuk
Copy link
Contributor

@mmatczuk mmatczuk commented Jan 3, 2024

Setting GOMEMLIMIT has very positive impact on the GC policy. It allows to defer GC til there is less work.
In my tests, using mainly local/browser test, the Heap usage is bellow 40MiB and the system allocated memory bellow 100MiB.

Setting GOMEMLIMIT has very positive impact on the GC policy.
It allows to defer GC til there is less work.
In my tests, using local/browser tests with MITM, the Heap usage is bellow 40MiB and the system allocated memory bellow 100MiB at all times.
Copy link
Contributor

@Choraden Choraden left a comment

Choose a reason for hiding this comment

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

LGTM.

We need to document it. How about we add a tunning section on the front page?

@mmatczuk mmatczuk merged commit de05143 into main Jan 5, 2024
@mmatczuk mmatczuk deleted the mmt/auto_gomemlimit branch January 5, 2024 10:08
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