-
Notifications
You must be signed in to change notification settings - Fork 743
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
Fix hanging of query when the key in order is an array #43427
Conversation
@poorna2152 Shall we add few test cases to oder by fields with type |
@@ -502,3 +502,98 @@ function incrementCount(int i) returns int { | |||
int count = i + 2; | |||
return count; | |||
} | |||
|
|||
function testQueryExprWithOrderByClauseWithArrayKey() { | |||
record {|int[] t; string s;|}[] data1 = [ |
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.
Lets add few more tests with record
type and enum
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.
It is not possible to use a record type as a key in the order by clause since it is not a ordered-type. https://ballerina.io/spec/lang/master/#ordering
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #43427 +/- ##
=========================================
Coverage ? 77.51%
Complexity ? 58594
=========================================
Files ? 3438
Lines ? 219234
Branches ? 28917
=========================================
Hits ? 169934
Misses ? 39880
Partials ? 9420 ☔ View full report in Codecov by Sentry. |
@poorna2152 shall we send the same fix for 2201.10.x and 2201.9.x branch as well? |
Purpose
Fixes #43418
Approach
Samples
Remarks
Check List