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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
