velox
velox copied to clipboard
facebookincubator
Add logic to serialize/deserialize SetAccumulators
This is the second in a set of PRs to add support for spilling distinct aggregations (see full version in https://github.com/facebookincubator/velox/pull/7791).
The logic to serialize/deserialize SetAccumulators is used in the DistinctAggregations for spilling.
Deploy Preview for meta-velox canceled.
| Name | Link |
|---|---|
| Latest commit | 9aa2cef2ebea0352be78d5e54a5983e38bc3335b |
| Latest deploy log | https://app.netlify.com/sites/meta-velox/deploys/6711400a1b3fe40008123db9 |
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
@mbasmanova : Have updated this code post rebase and addressing comments. PTAL.
@mbasmanova : Have updated the code to serialize to an ARRAY(VARBINARY) instead of a single String buffer. PTAL.
This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the PR, make sure you've addressed reviewer comments, and rebase on the latest main. Thank you for your contributions!
![stale[bot] avatar](https://avatars.githubusercontent.com/in/1724?v=4 "Published by a pub.dev verified publisher"){kind=small}
This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the PR, make sure you've addressed reviewer comments, and rebase on the latest main. Thank you for your contributions!