Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[POC] FieldSerializer for Records #931

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Conversation

theigl
Copy link
Collaborator

@theigl theigl commented Dec 16, 2022

This is a POC for moving record serialization from RecordSerializer into FieldSerializer.

This approach has two main advantages:

  1. Existing configuration options for the FieldSerializer can be used for records
  2. Sub-classes of FieldSerializer (Compatible, Versioned, Tagged, ..) can be adjusted for records

JDK 17+ is required for this PR so it is a likely candidate for the next major version.

@theigl theigl changed the title [POC] Record Field Serializer [POC] FieldSerializer for Records Dec 16, 2022
@theigl
Copy link
Collaborator Author

theigl commented Dec 21, 2022

See #884 for the discussion about RecordSerializer vs FieldSerializer for records.

…izer

# Conflicts:
#	test-jdk14/com/esotericsoftware/kryo/serializers/RecordSerializerTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant