RFR: 8321163: [test] OutputAnalyzer.getExitValue() unnecessarily logs even when process has already completed [v3]
Jaikiran Pai
jpai at openjdk.org
Fri Dec 1 12:44:17 UTC 2023
> Can I please get a review for this change to the test library's `OutputAnalyzer` class, which proposes to remove some unnecessary logging from the `getExitValue()` call?
>
> As noted in https://bugs.openjdk.org/browse/JDK-8321163, right now this method logs:
>
>
> [2023-11-24T11:47:54.557561Z] Waiting for completion for process 24909
> [2023-11-24T11:47:54.557873Z] Waiting for completion finished for process 24909
>
>
> even when the process has already completed and the exit value already known. The change in this PR makes it such that if the exit value is available then we no longer log this (nor call `process.waitFor()`).
>
> No new tests have been added given the nature of this change. tier1, tier2 and tier3 tests continue to pass with this change.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
remove micro optimization
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16919/files
- new: https://git.openjdk.org/jdk/pull/16919/files/2fc11922..d81f8e12
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16919&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16919&range=01-02
Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/16919.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16919/head:pull/16919
PR: https://git.openjdk.org/jdk/pull/16919
More information about the core-libs-dev
mailing list