-
Notifications
You must be signed in to change notification settings - Fork 48
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
chore(performance): use subresources & uriTemplate property to avoid … #4940
Conversation
…large unnecessary join tables
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.
i would also like to test it
@@ -13312,7 +13374,7 @@ components: | |||
items: | |||
$ref: '#/components/schemas/DayResponsible-read_Day.DayResponsibles' | |||
readOnly: true | |||
type: array | |||
type: unknown_type |
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.
what happened here?
⛔ Feature branch deployment currently inactive.If the PR is still open, you can add the |
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.
At least 25% better than before (with tiny requests)
…large unnecessary join tables
fixes #4927
The number of queries is a bit higher, but this avoids the large join table generated by the previous query.
Inspired by #3610 and by BacLuc#208