-
Couldn't load subscription status.
- Fork 11
Unnest on nested json arrays in flat collections #244
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #244 +/- ##
============================================
+ Coverage 80.08% 80.10% +0.01%
Complexity 1134 1134
============================================
Files 215 215
Lines 5484 5488 +4
Branches 481 482 +1
============================================
+ Hits 4392 4396 +4
Misses 764 764
Partials 328 328
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| Returns the column name with double quotes if the collection is flat to prevent folding to lower-case by PG | ||
| */ | ||
| private String getColName(boolean isFlatCollection, String pgColumnName) { | ||
| private String shouldQuoteColName(boolean isFlatCollection, String pgColumnName) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new method name seems like it would return a boolean.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah! Will fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Description
This PR adds support for
UNNESTon nester arrays in anyJSONBcolumn in flat collections.Checklist: