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

Jaikiran Pai jpai at openjdk.org
Fri Dec 1 01:34:05 UTC 2023


On Thu, 30 Nov 2023 16:23:39 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 incrementally with one additional commit since the last revision:
> 
>   Update OutputBuffer.java copyright years

These changes look good to me. Please wait for another review before integrating.

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

Marked as reviewed by jpai (Reviewer).

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


More information about the core-libs-dev mailing list