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.

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

Database query pipelining #219

Closed
aumetra opened this issue May 26, 2023 · 0 comments · Fixed by #229
Closed

Database query pipelining #219

aumetra opened this issue May 26, 2023 · 0 comments · Fixed by #229
Labels
enhancement New feature or request

Comments

@aumetra
Copy link
Member

aumetra commented May 26, 2023

With the migration to Diesel (#210), we gained the ability to pipeline queries. There are opportunities to utilise pipelining all over.
I tried to implement some in the initial release of the migration, but that ended in a bunch of bogus compile errors ("higher-ranked lifetime error"; possibly related to rust-lang/rust#102211).

We can check if we can work around these errors. If we can't, then this issue serves as a reminder to try again in a few releases.
@aumetra aumetra added the enhancement New feature or request label May 26, 2023
@aumetra aumetra mentioned this issue Jun 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Query pipelining kitsune-soc/kitsune
1 participant
@aumetra