-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
In multi-stage aggregate, directly store column index as identifier #11617
In multi-stage aggregate, directly store column index as identifier #11617
Conversation
e50179d
to
01c1200
Compare
Codecov Report
@@ Coverage Diff @@
## master #11617 +/- ##
============================================
- Coverage 63.08% 63.04% -0.05%
- Complexity 1106 1109 +3
============================================
Files 2326 2326
Lines 124942 124956 +14
Branches 19147 19152 +5
============================================
- Hits 78821 78777 -44
- Misses 40499 40555 +56
- Partials 5622 5624 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 16 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
01c1200
to
9f1fd3f
Compare
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.
lgtm. minor comment
arguments.add(ExpressionContext.forLiteralContext(literal)); | ||
continue; | ||
} | ||
if (i >= numArguments) { |
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.
intermediate agg always have 1 real-argument (it cannot even be 0 b/c count would've been converted to SUM), and the rest are placeholders, why do we need to check the numArguments again here? simply do the loop with for (int i = 1 ...
should be suffice?
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.
Separated the code path for raw input vs intermediate input
9f1fd3f
to
8c91320
Compare
Avoid constructing the column name to index map and converting back and forth