Skip to content

Delegating implementations of Session and StatelessSession #2238

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
Closed
gavinking opened this issue Apr 30, 2025 · 3 comments · Fixed by #2239 or #2246
Closed

Delegating implementations of Session and StatelessSession #2238

gavinking opened this issue Apr 30, 2025 · 3 comments · Fixed by #2239 or #2246
Labels
design A design or implementation issue
Milestone
3.0.0.CR1

Comments

@gavinking
Copy link
Member

It's going to be useful for quarkusio/quarkus#47462 to have MutinySessionDelegator and MutinyStatelessSessionDelegator base implementations like what we have in ORM. (I hate maintaining them, but I recognize their usefulness.)
@DavideD
Copy link
Member

DavideD commented Apr 30, 2025

They look very similar to MutinySessionImpl and MutinyStatelessImpl

gavinking added a commit to gavinking/hibernate-reactive that referenced this issue Apr 30, 2025
@gavinking
[hibernate#2238] create Delegators for us in Quarkus
dca55f8
gavinking added a commit to gavinking/hibernate-reactive that referenced this issue Apr 30, 2025
@gavinking
[hibernate#2238] create Delegators for use in Quarkus
dac4d59
gavinking added a commit to gavinking/hibernate-reactive that referenced this issue Apr 30, 2025
@gavinking
[hibernate#2238] create Delegators for use in Quarkus
1428690
gavinking added a commit to gavinking/hibernate-reactive that referenced this issue Apr 30, 2025
@gavinking
[hibernate#2238] create Delegators for use in Quarkus
5b6c0b7
@gavinking
Copy link
Member Author

They look very similar to MutinySessionImpl and MutinyStatelessImpl

I mean, no, not really...

gavinking added a commit that referenced this issue May 1, 2025
@gavinking
[#2238] create Delegators for use in Quarkus
d406878
@gavinking gavinking linked a pull request May 1, 2025 that will close this issue
[#2238] create Delegators for use in Quarkus #2239
Merged
@gavinking
Copy link
Member Author

Done.

gavinking added a commit that referenced this issue May 1, 2025
@gavinking
[#2238] create Delegators for use in Quarkus
a01cb1a
@DavideD DavideD linked a pull request May 5, 2025 that will close this issue
Remove MutinyStatelessSessionDelegator#upsert(String entityName, Object entity) method #2246
Merged
@DavideD DavideD added this to the 3.0.0.CR1 milestone May 5, 2025
@DavideD DavideD added the design A design or implementation issue label May 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
design A design or implementation issue
Projects
None yet
Milestone
3.0.0.CR1
2 participants
@DavideD @gavinking