Eevans has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/248960

Change subject: cassandra: updated gc settings
......................................................................

cassandra: updated gc settings

Updates GC settings for a 12G heap size, 8M region sizes, and an
initiating occupancy threshold of 35%.

See: https://phabricator.wikimedia.org/T106619#1604486

Note: This was applied once before as
https://gerrit.wikimedia.org/r/#/c/236391/4, but reverted because
the additional command arguments pushed the classname off the end,
wreaking havoc with Icinga.  Now that we are using systemd units
everywhere, this should no longer be an issue.

Bug: T106619
Change-Id: I9089354266e701be34faf2f9f066d0cb2c5e4224
---
M hieradata/role/common/cassandra.yaml
M modules/cassandra/templates/cassandra-env.sh.erb
2 files changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/60/248960/1

diff --git a/hieradata/role/common/cassandra.yaml 
b/hieradata/role/common/cassandra.yaml
index 02fc22d..48d713c 100644
--- a/hieradata/role/common/cassandra.yaml
+++ b/hieradata/role/common/cassandra.yaml
@@ -20,7 +20,7 @@
     - restbase2004.codfw.wmnet
     - restbase2005.codfw.wmnet
     - restbase2006.codfw.wmnet
-cassandra::max_heap_size: 16g
+cassandra::max_heap_size: 12g
 # 1/4 heap size, no more than 100m/thread
 cassandra::heap_newsize: 2048m
 cassandra::compaction_throughput_mb_per_sec: 60
diff --git a/modules/cassandra/templates/cassandra-env.sh.erb 
b/modules/cassandra/templates/cassandra-env.sh.erb
index b5e0083..9b80781 100644
--- a/modules/cassandra/templates/cassandra-env.sh.erb
+++ b/modules/cassandra/templates/cassandra-env.sh.erb
@@ -231,6 +231,8 @@
 JVM_OPTS="$JVM_OPTS -XX:+UseG1GC"
 JVM_OPTS="$JVM_OPTS -XX:G1RSetUpdatingPauseTimePercent=5"
 JVM_OPTS="$JVM_OPTS -XX:MaxGCPauseMillis=300"
+JVM_OPTS="$JVM_OPTS -XX:G1HeapRegionSize=8M"
+JVM_OPTS="$JVM_OPTS -XX:InitiatingHeapOccupancyPercent=35"
 
 # note: bash evals '1.7.x' as > '1.7' so this is really a >= 1.7 jvm check
 if [ "$JVM_ARCH" = "64-Bit" ] ; then

-- 
To view, visit https://gerrit.wikimedia.org/r/248960
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9089354266e701be34faf2f9f066d0cb2c5e4224
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Eevans <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to