[GitHub] [hadoop-ozone] fapifta edited a comment on issue #710: HDDS-3173. Provide better default JVM options

[GitBox <gi...@apache.org>]

fapifta edited a comment on issue #710: HDDS-3173. Provide better default JVM options
URL: https://github.com/apache/hadoop-ozone/pull/710#issuecomment-602717725
 
 
   I have one concern, in case there is any -XX option configured, then we skip to add GC options. This may be a problem in case when an extended JVM attribute is added for debug of performance purposes but one which is unrelated to GC, as in this case the GC behaviour and used algorithm may be altered unwillingly.
   
   It would be better to restrict the regex to something which is more close to GC stuff, like something that matches to GC case insensitive after a -XX but before a space... though I am unsure whether there is an ultimate solution for this, and maybe it is better to just document the suggested parameters for different jre versions, but we certainly should add some documentation on the potential behavioural change in case of JVM tuning somewhere.
   
   Also we talk about DataNodes where the problem happened, but the change seems to affect all daemonizable process types.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org