[GR-45044] Avoid emitting rmi library.

[graalvmbot]

It is not needed as it only provides an implementation for maxObjectInspectionAge(), which is substituted in SVM.

[fniephaus]

@roberttoyonaga could you take a look at this please?

[roberttoyonaga]

This looks good to me!

Can you please add RuntimeReflection.register(access.findClassByName("sun.rmi.server.UnicastRef2").getConstructors()); to the JMXClientFeature reflection config? It isn't related to your changes here, but is fixes a bug I caught while testing these changes.

You can also remove RuntimeReflection.register(access.findClassByName("sun.rmi.server.UnicastRef").getMethods()); I don't think its actually needed

[fniephaus]

Thanks for the quick review!

Can you please add RuntimeReflection.register(access.findClassByName("sun.rmi.server.UnicastRef2").getConstructors()); to the JMXClientFeature reflection config? It isn't related to your changes here, but is fixes a bug I caught while testing these changes.

Any way we can add a test for this?

[roberttoyonaga]

Any way we can add a test for this?

Hi Fabio, yes I can create a follow up PR with the test

[fniephaus]

Hi Fabio, yes I can create a follow up PR with the test

Sure, that'd work but then, I'd prefer if you also do the UnicastRef changes in that PR so that we have fix and test in one PR.

[roberttoyonaga]

I'd prefer if you also do the UnicastRef changes in that PR so that we have fix and test in one PR.

ok yes that makes sense