On Fri, 13 Mar 2026 12:46:13 GMT, Frederic Thevenet <[email protected]> 
wrote:

> When an `OutputAnalyzer` instance uses a `LazyOutputBuffer` impl to capture a 
> process' standard output, it unconditionally writes progress logs to stdout. 
> This can easily flood the output in tests that spawn a great number of 
> processes, each with an OutputAnalyzer attached, as part of their operations.
> 
> This change introduce an optional "quiet mode" which allows suppressing the 
> diagnostic messages emitted by the analyzer.
> 
> (Notes on implementation)
> * Introducing a "verbose mode" instead would likely have been more idiomatic, 
> but would have implied a lot of existing tests opting into this new mode to 
> retain their current printing behavior.
> * It is implemented as an extra parameter in new constructor overloads for 
> `OutputAnalyzer`, which really is the only way since `LazyOutputBuffer` 
> starts logging in its constructor.

Thank you for doing this!

-------------

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/30238#pullrequestreview-3944060039

Reply via email to