On Mon, 5 Oct 2026 17:02:16 GMT, Kevin Walls <[email protected]> wrote:

>> MBeanServer has a few methods called "deserialize", which are deprecated for 
>> almost 22 years (and "since 1.5").
>> The interface in MBeanServer.java implements these methods to throw 
>> UnsupportedOperationException by default.
>> 
>> JmxMBeanServer implements deserialize methods to call an MBeanInstantiator.  
>> These should be removed, and JmxMBeanServer will get the MBeanServer 
>> behaviour (UOE).  MBeanInstantiator (an internal class) should throw UOE 
>> directly.
>> 
>> There are no calls to these methods within the JDK or tests, except for one 
>> test: ArrayClassTest.  This exists to fix JDK-4974913 and even back in 2004 
>> this was known to be deprecated.  It should be updated to avoid the 
>> deprecated method.
>> 
>> 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Kevin Walls has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove deserialize, let UOE be thrown. Throw UOE directly in 
> MBeanInstantiator

LGTM

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

Marked as reviewed by dfuchs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/33214#pullrequestreview-5547556156

Reply via email to