Skip to content

Conversation

@gavinking
Copy link
Member

and clarify its semantics in the jdoc

[Please describe here what your change is about]


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Nov 20, 2024

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [ecb655f, 608b8b6]

› This message was automatically generated.

@gavinking gavinking changed the title correctly implement the documented semantics of beginTransaction() HHH-18867 correctly implement the documented semantics of beginTransaction() Nov 20, 2024
@gavinking
Copy link
Member Author

@sebersole WDYT?

@sebersole
Copy link
Member

I dislike changing the semantics here yet again. We've changed the semantics around Transaction a few times the last couple of releases.

But I agree this does match better with the JPA spec, so +1

…PA method

The JPA spec does not have anything to say about our beginTransaction() method
@gavinking gavinking merged commit 597f74f into hibernate:main Dec 16, 2024
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants