Skip to content

Revert "Add support for lambda traces via threadLocal"#6363

Closed
RanVaknin wants to merge 1 commit intomasterfrom
revert-6295-rvaknin/add-support-for-multi-concurrent-lambda-trace-id
Closed

Revert "Add support for lambda traces via threadLocal"#6363
RanVaknin wants to merge 1 commit intomasterfrom
revert-6295-rvaknin/add-support-for-multi-concurrent-lambda-trace-id

Conversation

@RanVaknin
Copy link
Contributor

Reverting the PR since slf4j implementation for MDC.get() is not provided by the SDK or by the Lambda runtime by default. This will only work if users provide their own implementation and it defeats the purpose of this working on Lambda by default.

@RanVaknin RanVaknin requested a review from a team as a code owner August 21, 2025 22:00
@sonarqubecloud
Copy link

@RanVaknin RanVaknin closed this Aug 22, 2025
@github-actions
Copy link

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant