On Thu, 5 Sep 2024 14:57:16 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Test
>
> Wow. I hadn't realized so many classes were impacted. I thought there was 
> just a few!
> 
> Look good in general. It could be good to write a test that sets the property 
> with 1.0 and 1.1, and that verifies that something (e.g. an ObjectName?) 
> serialized with property=1.0 can be deserialized with property=1.1, and 
> conversely.
> 
> That would validate that setting the property no longer has any effect.

Thanks Daniel @dfuch , we can use a version of what we had before, there was a 
simple check on ObjectName fields.  I added it, will fail in a JDK that 
respects jmx.serial.version.

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

PR Comment: https://git.openjdk.org/jdk/pull/20856#issuecomment-2332225753

Reply via email to