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
Open
Issue #8321 : Use the real Log4j2 SLF4J binding with a default config and publish Hop context as MDC#8374rmannibucau wants to merge 1 commit into
rmannibucau wants to merge 1 commit into
Conversation
…config and publish Hop context as MDC
Contributor
|
does this replace #8320 ? |
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 file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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:
mvn clean install apache-rat:checkto make sure basic checks pass. A more thorough check will be performed on your pull request automatically.git rebase -i.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.