RFR: 8332112: Update nsk.share.Log to don't print summary during VM shutdown hook [v2]

Leonid Mesnik lmesnik at openjdk.org
Tue May 14 22:19:19 UTC 2024


> The nsk.share.Log doing some cleanup and reporting errors in the cleanup method. This method is supposed to be executed by finalizer originally. However, now it is called only during shutdown hook. 
> The cleanup using Cleaner doesn't work. See https://bugs.openjdk.org/browse/JDK-8330760
> 
> The cleanup() method flush stream and print summary which should be already printed by complain method.
> 
> This cleanup is not necessary and printing summary usually is just disabled. It is enabled if the test called 'complain' method. However, the error should have been printed already in this method.
> 
> So it would be simple to remove this cleanup and reduce usage of Finalizable in vmTestbase tests.
> 
> Note: The 'verboseOnErrorEnabled' is just not used.
> 
> See isVerboseOnErrorEnabled.
> 
>     public boolean isVerboseOnErrorEnabled() {
>         return errorsSummaryEnabled;
>     }
> 
> 
> Tested with by running tests with different combinations (tier4-7) and tier1.

Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:

  fixed after comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/19209/files
  - new: https://git.openjdk.org/jdk/pull/19209/files/68b20e65..69ffd5b7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19209&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19209&range=00-01

  Stats: 27 lines in 25 files changed: 0 ins; 3 del; 24 mod
  Patch: https://git.openjdk.org/jdk/pull/19209.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19209/head:pull/19209

PR: https://git.openjdk.org/jdk/pull/19209


More information about the serviceability-dev mailing list