RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

Jaikiran Pai jpai at openjdk.org
Sat Jan 6 06:02:30 UTC 2024


On Fri, 5 Jan 2024 09:07:57 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Tests using ProcessTools.executeProcess gets the following output written to stdout:
>> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117
>> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117
>> [2023-11-24T09:58:23.071045055Z] Waiting for completion finished for process 2517117
>> 
>> This might be good for some use cases and debugging, but some tests spawns a large number of processes and for those this output fills up the log files.
>> 
>> I propose that we add a way to turn of this output for tests where we find this output to be too noisy.
>
> Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8320699_OutputAnalyzer_progress_logging
>  - Update OutputBuffer.java copyright years
>  - 8320699: Add parameter to skip progress logging of OutputAnalyzer

Still looks fine.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16807#pullrequestreview-1807268366


More information about the core-libs-dev mailing list