ManagedThreadFactory javadoc and spec discrepancy regarding ManageableThread

[glassfishrobot]

Section 3.4.4 of the spec says:
Threads returned by the newThread() method must implement the ManageableThread interface.

but javadoc says:
Threads returned from the newThread() method should implement the ManageableThread interface.

The spec is correct. The javadoc needs to be updated to say
Threads returned from the newThread() method must implement the ManageableThread interface.

Affected Versions

[1.0]

[glassfishrobot]

@glassfishrobot Commented
Reported by alai8

[glassfishrobot]

@glassfishrobot Commented
This issue was imported from java.net JIRA CONCURRENCY_EE_SPEC-32

[glassfishrobot]

• Issue Imported From: https://github.com/javaee/concurrency-ee-spec/issues/32
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To: @alai8

[njr-11]

The requirement to implement ManageableThread will not be achievable when creating virtual threads. Java requires a Thread.Builder to create virtual threads (or Thread.startVirtualThread which is defined as delegating to the builder via Thread.ofVirtual()) and seals the Thread.Builder interface to enforce that the only builders permitted are Thread.Builder.OfPlatform or Thread.Builder.OfVirtual which do not allow adding the ManageableThread interface to the Thread implementation class.

So in addition to fixing the existing inconsistency in the spec, we will also need to do something like qualifying the requirement as being for platform threads only.