You are viewing an old version of this page. View the current version.

Compare with Current View Page History

Version 1 Next »

This page is meant as a template for writing a KIP. To create a KIP choose Tools->Copy on this page and modify with your content and replace the heading with the next KIP number and a description of your issue. Replace anything in italics with your own description.

Status

Current state["Under Discussion"]

Discussion thread: here [Change the link from the KIP proposal email archive to your own email thread]

JIRA: here

Please keep the discussion on the mailing list rather than commenting on the wiki (wiki discussions get unwieldy fast).

Motivation

More discussions : https://github.com/apache/kafka/pull/16260#issuecomment-2159632052

Logging library used in tools module is inconsistent with core. slf4j2.x provides a cleaner and simpler approach.

By adding slf4j backends to dependencies, distributions will have their own slf4j backends, and its safe to define a slf4j provider in run-class. This allows to run kafka instance from source code with a specific provider too.

Public Interfaces

Update the below to upgrade slf4j

  • dependencies.gradle
  • build.gradle

Proposed Changes

1. Upgrade slf4j from 1.7.36 to 2.0.9+
2. Add a new system variable to run-class (sh & bat) script to define -Dslf4j.provider. By default we use org.slf4j.reload4j.Reload4jServiceProvider
3. Add other slf4j backend dependencies (optional)

Compatibility, Deprecation, and Migration Plan

Make sure users code works well when they provide their own slf4j jars

Test Plan

Describe in few sentences how the KIP will be tested. We are mostly interested in system tests (since unit-tests are specific to implementation details). How will we know that the implementation works as expected? How will we know nothing broke?

Rejected Alternatives

If there are alternative ways of accomplishing the same thing, what were they? The purpose of this section is to motivate why the design is the way it is and not some other way.

  • No labels