Skip to content

KAFKA-18752: Fix the warning of no-operation caused by slf4j in building kafka #19501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from

Conversation

frankvicky
Copy link
Contributor

@frankvicky frankvicky commented Apr 17, 2025

JIRA: KAFKA-18752 In KAFKA-18466, we remove the slf4j impl from runtime
scope to fix the compatibility issue (see
#17373 (comment)).
However, that also causes a side effect: when running some tasks, slf4j
can't find any provider as they are included in "release-only". We could
fix this by adding releaseOnly to the class path.

This patch leverages the task collection API to enable specific tasks to
access releaseOnly dependencies.

…ing kafka

JIRA: KAFKA-18752
In KAFKA-18466 we remove the slf4j impl from runtime scope to fix the
compatible issue (see
apache#17373 (comment)).
However, that also causes a side effect that in running some tasks slf4k
can't find any provider as they are included to "release-only"
We could fix this by adding releasOnly to class path
@frankvicky frankvicky requested a review from chia7712 April 17, 2025 06:21
@github-actions github-actions bot added triage PRs from the community build Gradle build or GitHub Actions small Small PRs labels Apr 17, 2025
@frankvicky
Copy link
Contributor Author

Build output:
image

Copy link
Collaborator

@m1a2st m1a2st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an typo in your description 'slf4k' should be 'slf4j'

@github-actions github-actions bot removed the triage PRs from the community label Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Gradle build or GitHub Actions small Small PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants