Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-4820

Slf4j / log4j version upgrade to support dynamic change of log levels --> Make logging framework exchangeable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: Build System
    • Labels:
      None

      Activity

      Hide
      greghogan Greg Hogan added a comment -

      Upgrade to what version? What are the implications of this upgrade?

      Show
      greghogan Greg Hogan added a comment - Upgrade to what version? What are the implications of this upgrade?
      Hide
      rmetzger Robert Metzger added a comment -

      Log4j 2 supports dynamic log level changes. I've filed a separate JIRA for adding support for Log4j2: FLINK-5339

      Show
      rmetzger Robert Metzger added a comment - Log4j 2 supports dynamic log level changes. I've filed a separate JIRA for adding support for Log4j2: FLINK-5339
      Hide
      StephanEwen Stephan Ewen added a comment -

      After a discussion with Robert Metzger, we decided to initially simply make Flink completely independent of a specific logging framework.

      We keep bundling log4j 1.x in flink-dist to not break all existing setups (log4j 2.x is config incompatible with log4j 1.x).

      Users can simply use log4j 2 instead by replacing the log4j-1.2.17.jar and slf4j-log4j12-1.7.7.jar with the log4j2 jar files (the same ways as using logback instead of log4j)

      Show
      StephanEwen Stephan Ewen added a comment - After a discussion with Robert Metzger , we decided to initially simply make Flink completely independent of a specific logging framework. We keep bundling log4j 1.x in flink-dist to not break all existing setups (log4j 2.x is config incompatible with log4j 1.x). Users can simply use log4j 2 instead by replacing the log4j-1.2.17.jar and slf4j-log4j12-1.7.7.jar with the log4j2 jar files (the same ways as using logback instead of log4j )
      Hide
      StephanEwen Stephan Ewen added a comment -

      Fixed in 478f5d1893e292ad68c2702cdae95771678dbdc2

      Show
      StephanEwen Stephan Ewen added a comment - Fixed in 478f5d1893e292ad68c2702cdae95771678dbdc2
      Hide
      StephanEwen Stephan Ewen added a comment -

      Zhenzhong Xu Does that work for you, or is something else still needed (like an slf4j upgrade)?

      Show
      StephanEwen Stephan Ewen added a comment - Zhenzhong Xu Does that work for you, or is something else still needed (like an slf4j upgrade)?

        People

        • Assignee:
          StephanEwen Stephan Ewen
          Reporter:
          zhenzhongxu Zhenzhong Xu
        • Votes:
          0 Vote for this issue
          Watchers:
          4 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development