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

disable gc_log_opts gc_tune options in favor of java_options #55

Open
vkhatri opened this issue Jul 24, 2015 · 1 comment
Open

disable gc_log_opts gc_tune options in favor of java_options #55

vkhatri opened this issue Jul 24, 2015 · 1 comment
Assignees
Milestone

Comments

@vkhatri
Copy link
Owner

vkhatri commented Jul 24, 2015

for solr v5.2.x comp we have added two new attribute default['solrcloud']['solr_config']['gc_log_opts'] and default['solrcloud']['solr_config']['gc_tune'] to the cookbook.

IMHO, it would be better to just use node['solrcloud']['java_options'] and drop both of them completely.

this provides simplicity.

creating this issue in case of any objection.

@vkhatri vkhatri self-assigned this Jul 24, 2015
@vkhatri vkhatri added this to the v5.2.1 milestone Jul 24, 2015
@vkhatri
Copy link
Owner Author

vkhatri commented Jul 24, 2015

looks like bin/solr has some hardcoded values for both GC_LOG_OPTS and GC_TUNE

if [ "$GC_LOG_OPTS" != "" ]; then
  GC_LOG_OPTS=($GC_LOG_OPTS "-Xloggc:$SOLR_LOGS_DIR/solr_gc.log")
  if [ "${JAVA_VERSION:0:3}" == "1.7" ]; then
    # Specific Java version hacking
    GC_TUNE+=('-XX:CMSFullGCsBeforeCompaction=1' '-XX:CMSTriggerPermRatio=80')
    JAVA_MINOR_VERSION=${JAVA_VERSION:(-2)}
    if [[ $JAVA_MINOR_VERSION -ge 40 && $JAVA_MINOR_VERSION -le 51 ]]; then
      GC_TUNE+=('-XX:-UseSuperWord')

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant