accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubbsii <>
Subject [GitHub] accumulo pull request #223: ACCUMULO-4409 Create AccumuloMonitorAppender
Date Wed, 01 Mar 2017 17:19:24 GMT
Github user ctubbsii commented on a diff in the pull request:
    --- Diff: assemble/conf/ ---
    @@ -13,43 +13,47 @@
     # See the License for the specific language governing permissions and
     # limitations under the License.
    -## Log4j 1.2 file that configures logging for all Accumulo services (Master, TabletServer,
GC & Tracer) except Monitor
    +## Log4j 1.2 file that configures logging for all Accumulo services (Master, TabletServer,
GC, and Tracer) except Monitor
     ## The system properties referenced below are configured by
    -## Write out INFO and higher to log file
    +## Define a log file appender
     log4j.appender.file.layout.ConversionPattern=%d{ISO8601} [%-8c{2}] %-5p: %m%n
    -## Forwards Accumulo messages at WARN or higher to Accumulo Monitor
    +## Define an appender to send important logs to the the primary Accumulo Monitor
    +## The primary monitor is the one currently holding a shared lock in ZooKeeper,
    +## and is typically the one that started first.
    -## Configures Audit logs which are OFF by default.
    +## Uncomment to define a log file appender for audit logs
    --- End diff --
    I'm also curious if there would have been problems with different processes writing to
the same file. I doubt the DailyRollingFileAppender used some sort of filesystem-independent
lock mechanism to avoid corruption.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message