Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Reporting range aggregate functions and IS JSON predicate clauses #2113
Reporting range aggregate functions and IS JSON predicate clauses #2113
Changes from all commits
56e5ab1
0d9261c
7995ae2
9785925
d465efa
4a0fdf6
2348e26
3700e44
6a96cdd
1bb6de4
078a6a1
5b1fe49
d5d2fd7
facaced
ea061d7
133bf75
bac4011
37f73b9
31d3d6b
db5e6e4
f954155
8b66a73
64b9c02
7144ffb
375c546
9c2e2f7
84a4a24
a944ba6
b6f3054
f3a8bd4
46b2f66
b4b8f54
734860d
d18c0ef
5546fad
1bc41ca
b285424
54aaa97
bed3120
df1420d
8d28862
1116517
c87cd8e
4183bd9
babfad0
f1f30ee
c93c816
1d666b1
29491dd
fccdd99
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Should we add separate tables for each of the IS JSON clauses (not even each but at least 2 or three different clauses in different tables)? Currently the tables might just be getting reported due to the first IS JSON clause and the rest seem kind of useless.
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.
Same as above, currently the view has multiple unsupported functions. Better to split it into multiple views to ensure that all the function types are getting caught.
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.
Same comment as above for this table too.