[GitHub] [geode] davebarnes97 commented on issue #4975: Logging documentation fixes

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

davebarnes97 commented on issue #4975:
URL: https://github.com/apache/geode/pull/4975#issuecomment-616863201


   Thanks for this contribution, @alb3rtobr. 
   For the gfsh command reference writeup on change loglevel, I'd like to suggest a minor re-wording, which replaces future tense with present tense.
   
   Now:
   Changes the logging level on specified members. This command will take effect if the default <%=vars.product_name%> logging configuration is used.
   
   In case of using custom `Log4J` configuration, this command will not work unless the member whose logging level you want to change was started using the `--J=-Dgeode.LOG_LEVEL_UPDATE_OCCURS=ALWAYS` system property.
   
   Suggested revision:
   Changes the logging level on specified members. This command takes effect if the default <%=vars.product_name%> logging configuration is used.
   
   When using a custom `Log4J` configuration, this command takes effect only if the member whose logging level you want to change was started using the `--J=-Dgeode.LOG_LEVEL_UPDATE_OCCURS=ALWAYS` system property.


----------------------------------------------------------------
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