-
Notifications
You must be signed in to change notification settings - Fork 3k
Document/test how to work with non-default PUs and Jakarta Data repositories #48877
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
Document/test how to work with non-default PUs and Jakarta Data repositories #48877
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
🙈 The PR is closed and the preview is expired. |
Thanks @marko-bekhta , this docs will help. FYI I have additional Quarkus tests for QE including DEV mode (and the DEV mode seems to work fine) quarkus-qe/quarkus-test-suite#2512 but it's definitely better that this project has own test coverage and documentation. |
} | ||
---- | ||
1. Pass the name of the non-default persistence unit to the repository annotation (`other` in this example). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1. Pass the name of the non-default persistence unit to the repository annotation (`other` in this example). | |
<1> Pass the name of the non-default persistence unit to the repository annotation (`other` in this example). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ohh 🙈 I copied the format from the example just above : https://github.com/marko-bekhta/quarkus/blob/37c0436fad343653ffe59234f5038f505f98859e/docs/src/main/asciidoc/hibernate-orm.adoc?plain=1#L1799-L1803 and guess who was the author of that 🫤 let me fix that other example as well... Thanks! 🙂
37c0436
to
5e60303
Compare
Status for workflow
|
Status for workflow
|
Awesome, thanks! |
I've added the info about the non-default PUs as a note, since it comes from the Jakarta Data and is not "unique" to Quarkus. + updated the data tests to use multiple PUs.