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

Surrealdb-orm: Implement chain query #259

Closed
Oyelowo opened this issue Mar 18, 2023 · 2 comments · Fixed by #279
Closed

Surrealdb-orm: Implement chain query #259

Oyelowo opened this issue Mar 18, 2023 · 2 comments · Fixed by #279
Assignees

Comments

@Oyelowo
Copy link
Owner

Oyelowo commented Mar 18, 2023

chain(query1)
   .chain(query2)
   .chain(query3);
@Oyelowo Oyelowo self-assigned this Mar 18, 2023
@Oyelowo
Copy link
Owner Author

Oyelowo commented Mar 19, 2023

I've decided this is totally unnecessary convenience function and can introduce more confusion. There is already support for transactions and if you ever have to run multiple queries concurrently, just use tokio or whatnot and spin off each query asynchronously. The goal of this project is to stay as idiomatic as possible and not introduce cruft with no significant upsides.

@Oyelowo Oyelowo closed this as completed Mar 19, 2023
@Oyelowo
Copy link
Owner Author

Oyelowo commented Mar 19, 2023

I take my word back. This may be useful e.g when you're using If else statement and you want multiple queries in a block or if youre doing schema definition and you just want to run all the queries without caring about the returned values

@Oyelowo Oyelowo pinned this issue Mar 19, 2023
@Oyelowo Oyelowo reopened this Mar 19, 2023
@Oyelowo Oyelowo linked a pull request Mar 19, 2023 that will close this issue
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 a pull request may close this issue.

1 participant