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

Backport PR #14431 to 8.4: [Doc] add off-heap memory session #14475

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

github-actions[bot]
Copy link
Contributor

Backport PR #14431 to 8.4 branch, original message:


Relates: #12622

Add a link to beats-input doc (logstash-plugins/logstash-input-beats#454)
Suggest to set -XX:MaxDirectMemorySize to half of the heap size

Relates: #12622

Co-authored-by: Karen Metts <[email protected]>
(cherry picked from commit 88d89fe)
@kaisecheng
Copy link
Contributor

github action closes the pr
reopen it

The operating system, persistent queue mmap pages, direct memory, and other processes require memory in addition to memory allocated to heap size.
Keep the overall memory requirements in mind when you allocate memory.

By default, a JVM's off-heap direct memory limit is the same as the heap size. Check out <<plugins-inputs-beats-memory,beats input memory usage>>.
Copy link
Contributor

@karenzone karenzone Aug 25, 2022

Choose a reason for hiding this comment

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

elastic/logstash-docs#1418 is a prereq to being able to use <<plugins-inputs-beats-memory,beats input memory usage>>. Docs CI will fail until #1418 gets merged.

@karenzone
Copy link
Contributor

@elasticmachine run elasticsearch-ci/docs

@karenzone
Copy link
Contributor

Looking promising! Waiting for the change to be push by the REAL doc build...

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Necessary files are in place to make this PR successful. LGTM!

@kaisecheng kaisecheng merged commit d4168bf into 8.4 Aug 25, 2022
@karenzone
Copy link
Contributor

karenzone commented Aug 25, 2022

Notes for future travelers:

Fortunately, those are on deck to be fixed soon.

@karenzone karenzone deleted the backport_14431_8.4 branch August 25, 2022 19:48
@roaksoax roaksoax added the docs label Dec 13, 2023
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