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