Skip to content

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

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
mamazzol:backport/9.2/pr-136152
Dec 8, 2025
Merged

[9.2] Remove trace logger in Transformer on default path when a class doesn't need to be transformed. (#136152)#139184
elasticsearchmachine merged 1 commit intoelastic:9.2from
mamazzol:backport/9.2/pr-136152

Conversation

@mamazzol
Copy link
Copy Markdown
Contributor

@mamazzol mamazzol commented Dec 8, 2025

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

…'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)
@mamazzol mamazzol requested a review from a team as a code owner December 8, 2025 11:46
@mamazzol mamazzol added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team labels Dec 8, 2025
@elasticsearchmachine elasticsearchmachine merged commit a3085e7 into elastic:9.2 Dec 8, 2025
34 checks passed
@mamazzol mamazzol deleted the backport/9.2/pr-136152 branch December 8, 2025 12:51
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 :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team v9.2.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants