On Fri, 14 Jun 2024 00:49:28 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> The nsk.share.Log has 3 parameters that might be configured by tests or >> using command-line: >> - verbose, traceLevel and timestamp >> >> The main purpose of these modes was to minimize output and use command-line >> arguments to enable them during bug reproducing/debugging for vmTestbase >> when it was compiled separately. >> >> However, such an approach has several disadvantages: >> -- For intermittent issues, there is no all data in the logs >> -- The enabling log might affect test behavior >> -- No easy way to set these command-line options for jtreg tests >> -- When verbose mode is disabled the messages are saved in some buffer and >> printed only test complains. The mode causes issues if the test fails >> without complaining (exception, crash, etc). The messages are just never >> printed. >> -- the solution is over-complicated. >> >> The fix enabled verbose mode and printing time stamps always, setting the >> debugging log level. >> >> The plan is to remove all these options and simplify logging as much as >> possible in the future. > > Leonid Mesnik has updated the pull request incrementally with one additional > commit since the last revision: > > reverted traceLevel test/hotspot/jtreg/vmTestbase/nsk/share/Log.java line 226: > 224: if (!verbose) { > 225: flushLogBuffer(); > 226: } flushLogBuffer() is never executed. I think you can remove it. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19613#discussion_r1639168255