Integrated: 8303773: Replace "main.wrapper" with "test.thread.factory" property in test code

Leonid Mesnik lmesnik at openjdk.org
Tue Oct 3 15:06:51 UTC 2023


On Wed, 27 Sep 2023 20:23:03 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 89987dbf
Author:    Leonid Mesnik <lmesnik at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/89987dbf8789f166fe24e7facf5b8cc9a3c292c2
Stats:     63 lines in 21 files changed: 0 ins; 1 del; 62 mod

8303773: Replace "main.wrapper" with "test.thread.factory" property in test code

Reviewed-by: cjplummer, amenkov

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

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


More information about the serviceability-dev mailing list