gnodet opened a new pull request, #12714: URL: https://github.com/apache/maven/pull/12714
## Summary - Migrates 70+ DEBUG log statements to TRACE in 6 areas that produce excessive noise under `-X`: lifecycle engine (reactor plan dumps, step scheduling), classrealm (realm creation/population/imports), cache internals (config resolution, access stats), resolver (descriptor filtering, relocation), model builder (cache clearing, profile activation), and plugin resolution (version/prefix tracing) - Adds `TRACE` to `Slf4jConfiguration.Level` enum so the logging system supports `-Dmaven.logger.defaultLogLevel=trace` as a system property for Maven core developers - `DEBUG` (`-X`) remains the right level for plugin development; `TRACE` is for Maven core developers diagnosing framework internals ## Motivation The TRACE level was added in the logging-foundation PR to separate two audiences: - **DEBUG** — plugin developers and power users diagnosing plugin behavior - **TRACE** — Maven core developers tracing internal framework mechanics Without this migration, TRACE exists but nothing emits at it, making the distinction theoretical. This PR populates it with the noisiest internal plumbing output that dominates `-X` today and drowns out the signal plugin developers actually want. ## What stays at DEBUG - Plugin parameter resolution, mojo configuration/loading (useful for plugin devs) - `mvnup` upgrade diagnostics - CLI bootstrap messages (uses `o.a.m.api.cli.Logger` which doesn't have TRACE) ## Files changed (18 files, +108/-105) | Area | Files | Changes | |---|---|---| | Lifecycle engine | `LifecycleDebugLogger`, `BuildPlanExecutor`, `MultiThreadedBuilder` | 32 statements | | Cache internals | `DefaultRequestCache`, `CacheConfigurationResolver` | 22 statements | | Classrealm | `DefaultClassRealmManager` | 8 statements | | Resolver | `DefaultArtifactDescriptorReader`, relocation sources, `TypeDeriver` | 8 statements | | Model builder | `DefaultModelBuilder` | 8 statements | | Plugin resolution | `DefaultPluginVersionResolver`, `DefaultPluginPrefixResolver` | 6 statements | | Infrastructure | `Slf4jConfiguration`, `MavenSimpleConfiguration`, `CliUtils` | TRACE level support | | Tests | `DefaultClassRealmManagerTest`, `LookupInvokerLoggingTest` | Updated mocks | ## Test plan - [x] All modified modules compile cleanly - [x] `DefaultClassRealmManagerTest` updated to verify `trace()` calls instead of `debug()` - [x] `LookupInvokerLoggingTest` updated for new TRACE enum value - [x] All tests pass in isolation (batch-mode flakiness in `MavenInvokerTest` is pre-existing) 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
