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
external_aggregation_fix for big endian machines #51280
external_aggregation_fix for big endian machines #51280
Conversation
@saitama951, you need to add s390x to CI to speed up the processing of your pull requests. |
@alexey-milovidov The team is working on the CI. |
This is an automated comment for commit 413ec52 with description of existing statuses. It's updated for the latest CI running
|
@alexey-milovidov fixed the code style issues. I think it's good to merge now. |
The external aggregation test was failing on s390x because incorrect hashing in TwoTevelStringAggregation.
Fix:
Changelog category (leave one):
Not For Changelog
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):