kryo icon indicating copy to clipboard operation
kryo copied to clipboard

Improve tests for JDK 17

Open theigl opened this issue 3 years ago • 2 comments

This PR attempts to fix as many tests as possible when running under JDK17 without any add-opens arguments.

Most tests work without opening up modules now. Only one tests still fails:

  • SerializationCompatTest fails because of AtomicInteger.

We need custom serializers for Atomic* JDK classes to fix this.

theigl avatar Feb 21 '22 14:02 theigl

I added a fall-back for lambda deserialization in f3cd2df.

theigl avatar Feb 21 '22 15:02 theigl

The only remaining issue is with AtomicInteger and we can't fix this without breaking backwards compatibility. We should add serializers for Atomic*, UUID, Pattern etc in Kryo 6.

theigl avatar Feb 21 '22 15:02 theigl