search

sous-chefs / kafka

Development repository for the kafka cookbook
https://supermarket.chef.io/cookbooks/kafka
Apache License 2.0
91 stars 105 forks source link

Java 10 compatibility #152

Closed fozboz closed 4 years ago

fozboz commented 6 years ago

The Java flag UseParNewGC has been removed from Java 10.

Also the logging functions have changed. To get this cookbook working, I had to set attributes as follows.

node.default['kafka']['jvm_performance_opts'] = %w[
  -server
  -XX:+UseCompressedOops
  -XX:+UseConcMarkSweepGC
  -XX:+CMSClassUnloadingEnabled
  -XX:+CMSScavengeBeforeRemark
  -XX:+DisableExplicitGC
  -Djava.awt.headless=true
].join(' ')

gc_log = ::File.join(node['kafka']['log_dir'], 'kafka-gc.log')

node.default['kafka']['gc_log_opts'] = %W[
  -Xlog:gc:file=#{gc_log}:time,uptime
].join(' ')

I think that gives us equivalent settings but I'm not a Java pro.

github-actions[bot] commented 4 years ago

Marking stale due to inactivity. Remove stale label or comment or this will be closed in 7 days. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.

github-actions[bot] commented 4 years ago

Closing due to inactivity. If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.

sglajchEG commented 2 years ago

This is still an issue with this cookbook. I had to apply the changes listed above to get kafka to run with JDK 11.