Skip to content

Issue #8321 : Use the real Log4j2 SLF4J binding with a default config and publish Hop context as MDC - #8374

Open
rmannibucau wants to merge 1 commit into
apache:mainfrom
rmannibucau:dev/log4j-slf4j-impl
Open

rmannibucau wants to merge 1 commit into
apache:mainfrom
rmannibucau:dev/log4j-slf4j-impl

Conversation

@rmannibucau

Copy link
Copy Markdown
Contributor

Follow up of #8321 as a primary releasable step.
Rest will likely pop up later in a discussion/issue - but might later this year to get back enough hop OSS time ot evaluate the options in all env.

Please add a meaningful description for your change here


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • [-] Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva

hansva commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

does this replace #8320 ?

@rmannibucau

Copy link
Copy Markdown
Contributor Author

@hansva it is more than this one has some agreement whereas the other didnt converge at all and this is already a huge step forward even if way less ambitious.

I'd keep the other opened mainly for history of the work with the note that we can need to take time to see if we make otel a first class citizen in hop superseeding current ad-hoc backbone and enabling the UI/server to integrate with an aggregator/collector rather than custom storage which doesn't work with all engines.
This can't be for a minor/patch release so this one is the intermediate step and we take time to refine a more global solution.
Means the other can be superseeded or not by another PR in a few months depending future discussions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants