Skip to content

Commit

Permalink
Remove double ES_JAVA_OPTS entry for heap size
Browse files Browse the repository at this point in the history
We don't need to pass Java memory parameters for the heap into multiple ES_ environment variables.

Fixes #434.
  • Loading branch information
martinb3 committed Feb 18, 2016
1 parent b54e079 commit 7c7b8f2
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions libraries/provider_configure.rb
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ def whyrun_supported?
params[:ES_JAVA_OPTS] << '-server '
params[:ES_JAVA_OPTS] << '-Djava.awt.headless=true '
params[:ES_JAVA_OPTS] << '-Djava.net.preferIPv4Stack=true '
params[:ES_JAVA_OPTS] << "-Xms#{new_resource.allocated_memory} "
params[:ES_JAVA_OPTS] << "-Xmx#{new_resource.allocated_memory} "
params[:ES_JAVA_OPTS] << "-Xss#{new_resource.thread_stack_size} "
params[:ES_JAVA_OPTS] << "#{new_resource.gc_settings.tr("\n", ' ')} " if new_resource.gc_settings
params[:ES_JAVA_OPTS] << '-Dfile.encoding=UTF-8 '
Expand Down

0 comments on commit 7c7b8f2

Please sign in to comment.