Skip to content

Fix JAVA_OPTS example in trace tuning docs - #7095

Open
martialblog wants to merge 1 commit into
opensearch-project:mainfrom
martialblog:docs/java-opts
Open

Fix JAVA_OPTS example in trace tuning docs#7095
martialblog wants to merge 1 commit into
opensearch-project:mainfrom
martialblog:docs/java-opts

Conversation

@martialblog

Copy link
Copy Markdown

Description

Hi,

The docs currently says:

Configure the Data Prepper heap by setting the JVM_OPTS environment variable.

I think that is wrong and needs to be JAVA_OPTS.

Issues Resolved

See #1021

Check List

  • New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Markus Opolka <markus.opolka@netways.de>
@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@srikanthpadakanti srikanthpadakanti left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, approving.

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