Kafka
  1. Kafka
  2. KAFKA-113

test log4j properties shouldn't be turned off

    Details

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

      Description

      core/src/test/resources/log4j.properties is set to off. We should change it to error. Also, this file doesn't seem to be picked up in unit test since there are lots of info level logging in the output.

      1. k113-v1.txt
        0.8 kB
        Chris Burroughs

        Activity

        Jun Rao created issue -
        Hide
        Chris Burroughs added a comment -
        • Turns logging for tests on by default
        • I used WARN, that appears to result in very little output
        • Adds line numbers
        • For me setting log4j.logger.kafka=TRACE results in a torrent of output.
        Show
        Chris Burroughs added a comment - Turns logging for tests on by default I used WARN, that appears to result in very little output Adds line numbers For me setting log4j.logger.kafka=TRACE results in a torrent of output.
        Chris Burroughs made changes -
        Field Original Value New Value
        Attachment k113-v1.txt [ 12491356 ]
        Chris Burroughs made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Assignee Chris Burroughs [ cburroughs ]
        Fix Version/s 0.7 [ 12317243 ]
        Hide
        Jun Rao added a comment -

        +1. Thanks for the patch, Chris.

        Show
        Jun Rao added a comment - +1. Thanks for the patch, Chris.
        Chris Burroughs made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Chris Burroughs
            Reporter:
            Jun Rao
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development