On Thu, 4 Jan 2024 02:13:18 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> Remove the MLet feature and its tests.
>> 
>> Some tests use MLet classes but are not testing MLets.  These are updated, 
>> to use another test MBean or an MBean which is a URLClassLoader, e.g.
>> test/jdk/javax/management/MBeanServer/PostExceptionTest.java
>> test/jdk/javax/management/remote/mandatory/loading/TargetMBeanTest.java
>
> test/jdk/javax/management/remote/mandatory/connection/IdleTimeoutTest.java 
> line 214:
> 
>> 212:                                        +"]: starting at " +
>> 213:                                        elapsed + "ms");
>> 214:                     final String name = ":name=Test_instance_" + i;
> 
> Q: The same question, why it is not started with the `d:type=` as it was 
> before?

Yes, just an identifier, the content is not very important.
type=mlet should be removed to clarify it's not an m-let, but would not stop it 
from working.
I could leave this one unchanged other than removing "type=mlet".

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16363#discussion_r1441585077

Reply via email to