-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
Refactor array, structured and JSON expressions. #929
Conversation
…ExpressionBuilder`. Refactor `ArrayExpression`, `StructuredExpression` and `JsonExpression`. Implement lazy arrays.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #929 +/- ##
============================================
- Coverage 99.62% 99.26% -0.36%
- Complexity 1490 1539 +49
============================================
Files 81 98 +17
Lines 3689 3805 +116
============================================
+ Hits 3675 3777 +102
- Misses 14 28 +14 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Does it affect docs in any way? |
Overall 👍 |
Co-authored-by: Alexander Makarov <[email protected]>
It adds new features. I plan to add the docs later, after supporting custom type-casting. |
ArrayExpressionBuilder
,StructuredExpressionBuilder
andJsonExpressionBuilder
;ArrayExpression
,StructuredExpression
andJsonExpression
;db-sqlite
,db-mysql
,db-pgsql
.Related PRs
JsonExpressionBuilder
and JSON type tests db-sqlite#337JsonExpressionBuilder
and JSON type tests db-mysql#376