Skip to content

[8.19] Remove trace logger in Transformer on default path when a class doesn't need to be transformed. (#136152)#143138

Merged
elasticsearchmachine merged 2 commits intoelastic:8.19from
mosche:backport/8.19/pr-136152
Feb 26, 2026
Merged

[8.19] Remove trace logger in Transformer on default path when a class doesn't need to be transformed. (#136152)#143138
elasticsearchmachine merged 2 commits intoelastic:8.19from
mosche:backport/8.19/pr-136152

Conversation

@mosche
Copy link
Copy Markdown
Contributor

@mosche mosche commented Feb 26, 2026

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

Fixes #142635

…'t need to be transformed. (elastic#136152)

This trace log badly interfered with entitlements in tests. When a class (eg Regex, used by the test mock logger in the case of JULBridgeTests) is loaded as a result of logging, tracing logging that in the Transformer causes a recursive call to the logging appender.

Fixes elastic#132280

(cherry picked from commit 2858d6a)

# Conflicts:
#	muted-tests.yml
@mosche mosche added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 26, 2026
@elasticsearchmachine elasticsearchmachine merged commit e7e122e into elastic:8.19 Feb 26, 2026
28 checks passed
@mosche mosche deleted the backport/8.19/pr-136152 branch February 26, 2026 18:01
neilbhavsar pushed a commit that referenced this pull request Feb 27, 2026
…'t need to be transformed. (#136152) (#143138)

This trace log badly interfered with entitlements in tests. When a class (eg Regex, used by the test mock logger in the case of JULBridgeTests) is loaded as a result of logging, tracing logging that in the Transformer causes a recursive call to the logging appender.

Fixes #132280

(cherry picked from commit 2858d6a)

# Conflicts:
#	muted-tests.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.19.13

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants