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
Flat hash aggregation #207
Conversation
return binder.bind(inputs); | ||
if (inputs.isEmpty()) { | ||
return aggregation(function, -1); | ||
} else { |
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.
formatting
Add back vectorized aggergation operation
slice.setByte(index, slice.getByte(index) | bitMask); | ||
} | ||
|
||
public void setNotNull(Slice slice, int offset, int field) |
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.
I think it's a bit weird that a field can be marked as null/notnull independently from setting its value. Calling setLong/Double/etc should make the field non-null. Clearly, doing this on every call to those methods would impact performance, so I don't know if there's a better approach.
} | ||
|
||
return PageIterators.singletonIterator(new Page(blocks)); | ||
} | ||
|
||
@SuppressWarnings("rawtypes") | ||
public static Aggregator createAggregator(AggregationFunctionDefinition functionDefinition, Step step) |
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.
Mark this @VisibleForTesting
and maybe make it package private
looks good |
No description provided.