Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-1242 Consistent logging
  3. ACCUMULO-3651

Replace log4j dependencies with slf4j where setLevel is not used.

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.7.0
    • 1.7.0
    • build

    Description

      Because of the issues originally encountered where setLevel is used, I'm going to break the task into a few sub-tasks that can be accomplished in stages.

      This is the first sub-task and will include converting to slf4j where setLevel is not used (that will be handled separately, after more discussion) nor where dom manipulation is used (tests if memory serves.)

      This sub-task should allow us to move forward while deferring the harder situations for later (and in other sub-tasks.)

      The work on this task will include some modification to log statements where exceptions are thrown without a message (message is required by slf4j) This sub-task will not include conversion to use {} parameter formatting optimization to avoid string concatenation - that will be a follow on sub-task.

      My rational for multiple sub-tasks it to allow the changes to be evaluated in isolation so that if there are objections, the core issue of removing direct log4j dependencies can continue to move forward.

      Attachments

        1. ACCUMULO-3651.patch
          322 kB
          Ed Coleman
        2. ACCUMULO-3651.2.patch
          322 kB
          Ed Coleman

        Activity

          People

            etcoleman Ed Coleman
            etcoleman Ed Coleman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 10m
                10m