search

geerlingguy / ansible-role-elasticsearch

Ansible Role - Elasticsearch
https://galaxy.ansible.com/geerlingguy/elasticsearch/
MIT License
185 stars 209 forks source link

Replace min/max heap with Single `elasticsearch_heap_size` variable #63

Closed cyntheticfox closed 4 years ago

cyntheticfox commented 4 years ago

Replace minimum and maximum heap size declarations with single declaration of elasticsearch_heap_size.

This replacement is to better line up with both the given comments in the jvm.options.j2 template stating that maximum heap size must equal minimum heap size, as well as to match the official elastic repository. Variability between the two can result in elasticsearch either not running or instability.

Closes #62

stale[bot] commented 4 years ago

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

stale[bot] commented 4 years ago

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.