RFR: 8303773: Replace "main.wrapper" with "test.thread.factory" property in test code [v2]
Leonid Mesnik
lmesnik at openjdk.org
Thu Sep 28 16:47:09 UTC 2023
> The main.wrapper was the first name for jtreg test thread factory plugin. However, during integration of this feature in jtreg it was decided to use test.thread.factory name. So this fix just renames "main.wrapper" property to "test.thread.factory" so it is more compliant with jtreg naming. Also, it makes more sense for tests when it is used to create other then main threads in test.
> Testing: tier1-5.
> Verified that "main.wrapper" is not used in test sources anymore.
>
> I haven't rename DebugeeWrapperd and MainWrapper classes in JDI test frameworks because they are actually more main wrappers than thread factories.
Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
fix
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15950/files
- new: https://git.openjdk.org/jdk/pull/15950/files/97e4ae30..896065e5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=00-01
Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/15950.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/15950/head:pull/15950
PR: https://git.openjdk.org/jdk/pull/15950
More information about the serviceability-dev
mailing list