-
Notifications
You must be signed in to change notification settings - Fork 40
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
Immutable queries? #71
Comments
No, the idea was to always use queries inline for readability. |
Would you consider adding such a feature, under a different method name? (PR #72) |
@cassiozen +1 |
That seems like a very bad excuse. When building a data access layer it is very common to share queries, especially CTEs. Adding something like #72 also doesn't remove the ability to use queries inline. |
I suggest |
Currently
query.append
mutates the original query, which makes cumbersome to reuse a base query in multiple places.For example, this will not work - Every time the route handler gets called, a new WHERE segment will be appended to the original baseQuery:
Is there any immutable way of appending?
The text was updated successfully, but these errors were encountered: