RFR: 8333270: HandlersOnComplexResetUpdate and HandlersOnComplexUpdate tests fail with "Unexpected reference" if timeoutFactor is less than 1/3 [v2]
Daniel Fuchs
dfuchs at openjdk.org
Tue Jun 4 15:17:33 UTC 2024
> HandlersOnComplexResetUpdate and HandlersOnComplexUpdate tests verify that loggers are GC'ed (or not GC'ed) after a reset or an update. For that they poll a ReferenceQueue in a loop. The number of iteration is adjusted according to the jtreg timeout factor. However, the logic in the test did not expect that the timeout might be less than 1.
>
> This fix does two things:
>
> 1. fix the adjustCount logic - so that the number of iteration can only be increased
> 2. use remove(timeout) instead of poll() in order to minimize the waiting time.
Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
Review feedback: use Reference::refersTo consistently
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19503/files
- new: https://git.openjdk.org/jdk/pull/19503/files/be5a4d82..75576a8d
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19503&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19503&range=00-01
Stats: 5 lines in 2 files changed: 0 ins; 0 del; 5 mod
Patch: https://git.openjdk.org/jdk/pull/19503.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19503/head:pull/19503
PR: https://git.openjdk.org/jdk/pull/19503
More information about the core-libs-dev
mailing list