Accumulo
  1. Accumulo
  2. ACCUMULO-2383

Allow configuration of logging using properties file also

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.1, 1.7.0
    • Component/s: None
    • Labels:
      None

      Description

      We currently read only XML files to be used for configuring log4j in Accumulo. We should allow properties file too for that.

        Issue Links

          Activity

          Hide
          ASF subversion and git services added a comment -

          Commit cdcc1d4d41bf16e628b87893814a0ae9f627bcdb in accumulo's branch refs/heads/master from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=cdcc1d4 ]

          ACCUMULO-2383 Add log4j properties support

          Accumulo now looks for either XML or properties files for the Log4J configuration. It will
          also heed the log4j.configuration isystem property if it is set. Its MonitorLog4jWatcher
          can now load either properties or XML.

          Show
          ASF subversion and git services added a comment - Commit cdcc1d4d41bf16e628b87893814a0ae9f627bcdb in accumulo's branch refs/heads/master from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=cdcc1d4 ] ACCUMULO-2383 Add log4j properties support Accumulo now looks for either XML or properties files for the Log4J configuration. It will also heed the log4j.configuration isystem property if it is set. Its MonitorLog4jWatcher can now load either properties or XML.
          Hide
          ASF subversion and git services added a comment -

          Commit cdcc1d4d41bf16e628b87893814a0ae9f627bcdb in accumulo's branch refs/heads/1.6.1-SNAPSHOT from Bill Havanki
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=cdcc1d4 ]

          ACCUMULO-2383 Add log4j properties support

          Accumulo now looks for either XML or properties files for the Log4J configuration. It will
          also heed the log4j.configuration isystem property if it is set. Its MonitorLog4jWatcher
          can now load either properties or XML.

          Show
          ASF subversion and git services added a comment - Commit cdcc1d4d41bf16e628b87893814a0ae9f627bcdb in accumulo's branch refs/heads/1.6.1-SNAPSHOT from Bill Havanki [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=cdcc1d4 ] ACCUMULO-2383 Add log4j properties support Accumulo now looks for either XML or properties files for the Log4J configuration. It will also heed the log4j.configuration isystem property if it is set. Its MonitorLog4jWatcher can now load either properties or XML.
          Hide
          Bill Havanki added a comment -

          Forgot to link review and update ticket status the other day - doing so now. Review is already in submittable state.

          Show
          Bill Havanki added a comment - Forgot to link review and update ticket status the other day - doing so now. Review is already in submittable state.
          Hide
          Sean Busbey added a comment -

          Best to leave the legacy example configs as-is, I think.

          Show
          Sean Busbey added a comment - Best to leave the legacy example configs as-is, I think.
          Hide
          Bill Havanki added a comment -

          I have this working based on ACCUMULO-2343. I've developed generic_logger.properties and monitor_logger.properties files to include in the source tree. I expect to only include them in conf/templates, but if anyone would like, I can include them in the conf/examples directories as well.

          Show
          Bill Havanki added a comment - I have this working based on ACCUMULO-2343 . I've developed generic_logger.properties and monitor_logger.properties files to include in the source tree. I expect to only include them in conf/templates, but if anyone would like, I can include them in the conf/examples directories as well.
          Hide
          Vikram Srivastava added a comment -

          AsyncAppender isn't supported for properties file. So the new Appender is a pre-requisite if we want to have equivalent functionality as xml files.

          Show
          Vikram Srivastava added a comment - AsyncAppender isn't supported for properties file. So the new Appender is a pre-requisite if we want to have equivalent functionality as xml files.

            People

            • Assignee:
              Bill Havanki
              Reporter:
              Vikram Srivastava
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development