RFR: 8303703: Add support of execution tests using virtual thread factory jtreg plugin

Leonid Mesnik lmesnik at openjdk.org
Tue Apr 18 14:46:48 UTC 2023


On Tue, 18 Apr 2023 05:59:56 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> test/jtreg_test_thread_factory/src/share/classes/Virtual.java line 46:
>> 
>>> 44:     public Thread newThread(Runnable task) {
>>> 45:         try {
>>> 46:             return Thread.ofVirtual().factory().newThread(task);
>> 
>> As far as I can see, none of these method calls throw a checked exception. Should we perhaps remove this try/catch block and let any runtime exception that gets thrown be propagated as-is instead of wrapping it in a `RuntimException`?
>
> It's also a bit strange that it creates a new ThreadFactory each time, I had expected to be a wrapper around one ThreadFactory. The effect is the same, just a bit strange.
> 
> (I suspect the try-catch dates from when it called Thread.ofVirtual with reflection).

I've updated it to reuse thread factory and don't check exceptions.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13432#discussion_r1170148956



More information about the build-dev mailing list