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.

Sign up for GitHub

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

Jump to bottom

Allow Java 16 records to be used for @PrimaryKeyClass #1126

Closed
mp911de opened this issue May 5, 2021 · 0 comments
Closed

Allow Java 16 records to be used for @PrimaryKeyClass #1126

mp911de opened this issue May 5, 2021 · 0 comments
Labels
type: enhancement A general enhancement
Milestone
3.1.9 (2020.0.9)

Comments

@mp911de
Copy link
Member

mp911de commented May 5, 2021

PrimaryKeyClassEntityMetadataVerifier checks whether the superclass is Object. We could drop this validation altogether because it is overly restricting.
@mp911de mp911de added the type: enhancement A general enhancement label May 5, 2021
mp911de added a commit that referenced this issue May 6, 2021
@mp911de
PrimaryKeyClass can now subclass other types than Object.
fbbcec9 
We dropped the requirement that composite keys may only subclass Object to allow usage with other superclasses such as Java records.

Closes #1126
mp911de added a commit that referenced this issue May 6, 2021
@mp911de
PrimaryKeyClass can now subclass other types than Object.
dff3c4e 
We dropped the requirement that composite keys may only subclass Object to allow usage with other superclasses such as Java records.

Closes #1126
@mp911de mp911de closed this as completed in a60b2c4 May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
3.1.9 (2020.0.9)
Development

No branches or pull requests
1 participant
@mp911de