Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-3546

Log4j should be initialized using java system properties, not code

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0
    • None
    • None

    Description

      Our log4j configuration files are confusing. It is not clear, unless looking at code, how our configuration files for logging are supposed to be used, and for which services.

      Even then, there are bugs, such as ACCUMULO-3544, because we assume we can log normally without first initializating the logging system.

      We should rely on log4j's default behavior to initialize the logging based on standard configuration files. If we need to specify a different file, we should do that with the java system properties that log4j supports (-Dlog4j=...) and set those in our environment scripts.

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. ACCUMULO-2349 Accumulo should give some notification if there is no *_logger.xml file

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. ACCUMULO-3544 accumulo-start uses log4j before it is initialized

          • Major - Major loss of function.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. ACCUMULO-3955 switch off of log4j v1.x

          • Major - Major loss of function.
          • Resolved
          supercedes

          Task - A task that needs to be done. ACCUMULO-2954 Clarify properties vs. xml precedence for generic_logger and monitor_logger

          • Major - Major loss of function.
          • Resolved

          Activity

            People

              Unassigned Unassigned
              ctubbsii Christopher Tubbs
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: