Skip to content
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

Improve UploaderInfo with some JOINs #185

Open
ro-tex opened this issue Apr 5, 2022 · 1 comment
Open

Improve UploaderInfo with some JOINs #185

ro-tex opened this issue Apr 5, 2022 · 1 comment

Comments

@ro-tex
Copy link
Contributor

ro-tex commented Apr 5, 2022

f/u

Definitely not a requirement for this PR but I wonder if PJ could turn this into some fancy query which fetches all uploads for a skylink id and then joins them with their users where possible. So that all of that happens with a single query.

Originally posted by @ChrisSchinnerl in #182 (comment)

f/u Might be a tiny bit more efficient to use a map to pointers to avoid this dereference here. Because the ID, Email, Sub and StripeID are later copied again anyway.

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

No branches or pull requests

1 participant