jakartaee / jakartaee/persistence

Remove declaration of ResultSetMapping as sealed

Open
#887 0 comments 0 reactions 0 assignees Claimed by @sebersole View on GitHub
Dominant language
Java
Stars
268
Forks
78
Avg merge
1d 6h
Merged PRs (30d)
13

Description

Hibernate, and I am assuming other providers, have historical ways to define more rich ResultSet mappings.

Because ResultSetMapping was introduced as sealed and all of its impls as records, it is impossible to model those richer models using this API; which is a shame imo. Here is a PR to remove the seal and allow such usage.

A further step would be to make the impls NOT records. This would allow even better implementation of this idea by implementors. I held off on that for the time being until we had consensus on this idea in general.

Contributor guide

Open the contributing guide

Research direction

Review the linked pull request for the proposed ResultSetMapping API change and inspect the current sealed declaration and its record implementations. The immediate scope is to allow provider-defined implementations by removing the seal; changing the implementations from records is explicitly deferred. Done means the API accepts richer provider-specific mappings without taking on that follow-up.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend-api-design, databases
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.