[GitHub] [accumulo-website] ctubbsii commented on a diff in pull request #378: Fix broken links to Log config files

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii commented on code in PR #378:
URL: https://github.com/apache/accumulo-website/pull/378#discussion_r1131087234


##########
_docs-2/configuration/files.md:
##########
@@ -38,16 +38,16 @@ Accumulo processes. See the [quick start] for help with configuring this file.
 
 ### log4j-service.properties
 
-The {% ghc assemble/conf/log4j-service.properties %} file configures logging for most Accumulo services
+The {% ghc assemble/conf/log4j2-service.properties %} file configures logging for most Accumulo services
 (i.e [Manager], [Tablet Server], [Garbage Collector]) except for the Monitor.
 
 ### log4j-monitor.properties
 
-The {% ghc assemble/conf/log4j-monitor.properties %} file configures logging for the [Monitor].
+The `log4j2-monitor.properties` file configures logging for the [Monitor].

Review Comment:
   The monitor does not have, or require, a separate configuration file. It just uses the normal `log4j-service.properties` file. This was probably different in earlier versions, but without going back and investigating, I can't remember the details off the top of my head. We can just remove this description here, or replace it with an explanation of how one can configure the `log4j-service.properties` to forward to the monitor appender (which should already exist by default in that config file).



##########
_docs-2/configuration/files.md:
##########
@@ -38,16 +38,16 @@ Accumulo processes. See the [quick start] for help with configuring this file.
 
 ### log4j-service.properties

Review Comment:
   The names of these headers no longer match the file names in the config directory. They should probably be updated as well.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org