[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8303773: Replace "main.wrapper" with "test.thread.factory" property in test code [v3]
From:       Leonid Mesnik <lmesnik () openjdk ! org>
Date:       2023-09-29 18:54:21
Message-ID: DHPd0IXCLNc2UoSQyKMphfNdGrYNLhc9fbM9miwulgM=.9672332b-e4f2-4d92-9e40-54c45e9b983d () github ! com
[Download RAW message or body]

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

  spaces added.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15950/files
  - new: https://git.openjdk.org/jdk/pull/15950/files/896065e5..284a341d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=01-02

  Stats: 3 lines in 3 files changed: 0 ins; 0 del; 3 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


[prev in list] [next in list] [prev in thread] [next in thread]