RFR: 7904088: Include the timestamp to the log message when a timeout is fired
Jaikiran Pai
jpai at openjdk.org
Fri Sep 26 07:06:15 UTC 2025
Can I please get a review of this trivial change which includes the timestamp for a log message when a test action times out?
Before this change, jtreg would log a message like:
> Timeout signalled after 120 seconds
Now it will also include a timestamp for that log message. This should help during investigations of timing out tests:
> [08:10:07.344] Timeout signalled after 120 seconds
The change is a one liner in `src/share/classes/com/sun/javatest/regtest/agent/Alarm.java`. Rest of the changes are mere refactoring to allow for using the same `DateTimeFormat` consistently in a couple of other places.
-------------
Commit messages:
- add timestamp to log
Changes: https://git.openjdk.org/jtreg/pull/293/files
Webrev: https://webrevs.openjdk.org/?repo=jtreg&pr=293&range=00
Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904088
Stats: 67 lines in 5 files changed: 48 ins; 15 del; 4 mod
Patch: https://git.openjdk.org/jtreg/pull/293.diff
Fetch: git fetch https://git.openjdk.org/jtreg.git pull/293/head:pull/293
PR: https://git.openjdk.org/jtreg/pull/293
More information about the jtreg-dev
mailing list