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