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

        Hide
        Jun Rao added a comment -

        +1. Thanks for the patch, Chris.

        Show
        Jun Rao added a comment - +1. Thanks for the patch, Chris.
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development