Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

heap.dump.path is set as mandatory variable. Setting it to optional i… #668

Closed
wants to merge 2 commits into from
Closed

heap.dump.path is set as mandatory variable. Setting it to optional i… #668

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f4b6e05
heap.dump.path is set as mandatory variable. Setting it to optional i…
ozonsozons Jan 28, 2020
ec097e8
Merge branch 'master' into fix-heap-dump-path-variable
jmlrt Mar 4, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion templates/jvm.options.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,9 @@

# specify an alternative path for heap dumps; ensure the directory exists and
# has sufficient space
-XX:HeapDumpPath=${heap.dump.path}
{% if heap.dump.path is defined %}
-XX:HeapDumpPath=${heap.dump.path}
{% endif %}
Comment on lines +113 to +115
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{% if heap.dump.path is defined %}
-XX:HeapDumpPath=${heap.dump.path}
{% endif %}
${heap.dump.path}


# specify an alternative path for JVM fatal error logs
-XX:ErrorFile={{ es_log_dir }}/hs_err_pid%p.log
Expand Down