michaelsembwever commented on code in PR #3696: URL: https://github.com/apache/cassandra/pull/3696#discussion_r2725803681
########## conf/jvm-server.options: ########## @@ -185,6 +185,25 @@ # For production use you may wish to adjust this for your environment. # If that's the case, see MAX_HEAP_SIZE (and HEAP_NEWSIZE for CMS) in cassandra-env.sh +# Need experimental bytebuddy for JDK21 +-Dnet.bytebuddy.experimental + +# Young generation size is automatically calculated by cassandra-env +# based on this formula: min(100 * num_cores, 1/4 * heap size) +# +# The main trade-off for the young generation is that the larger it +# is, the longer GC pause times will be. The shorter it is, the more +# expensive GC will be (usually). +# +# It is not recommended to set the young generation size if using the +# G1 GC, since that will override the target pause-time goal. +# More info: http://www.oracle.com/technetwork/articles/java/g1gc-1984535.html +# +# The example below assumes a modern 8-core+ machine for decent +# times. If in doubt, and if you do not particularly want to tweak, go +# 100 MB per physical CPU core. +#-Xmn800M Review Comment: this also exists here: https://github.com/jmckenzie-dev/cassandra/blob/1564529f5b7dce3a20f10dfd0add2f146b3991de/conf/jvm11-server.options#L53-L58 where it is better placed, being only relevant to CMS. GenZGC does not have the young/old distinction, and G1 uses `-XX:G1NewSizePercent=50` here: https://github.com/jmckenzie-dev/cassandra/blob/1564529f5b7dce3a20f10dfd0add2f146b3991de/conf/jvm11-server.options#L70C1-L70C24 i think what's highlighted here can simply be deleted. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
