velox
velox copied to clipboard
Add array_cum_sum function
Adds Presto array function array_cum_sum
. Resolves https://github.com/prestodb/presto/issues/20293 .
Deploy Preview for meta-velox canceled.
Name | Link |
---|---|
Latest commit | ea6431f073d5cbb844813eda3beadef65ca9fdad |
Latest deploy log | https://app.netlify.com/sites/meta-velox/deploys/6668cee90709070007123fee |
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!
@pramodsatya : Please can you rebase and get a clean test run.
@Yuhta @kagamiori : Would be great to get your review/merge help.
Hi @pramodsatya it looks like we lost track of this change as it was never merged, could you rebase and update the PR?
Hi @pramodsatya it looks like we lost track of this change as it was never merged, could you rebase and update the PR?
Hi @kevinwilfong, I have rebased the PR, thanks for helping with the merge.
@kevinwilfong has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@kevinwilfong has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@kevinwilfong merged this pull request in facebookincubator/velox@79bc73def5622d4659cde46f4249fef3e45726c9.
Conbench analyzed the 1 benchmark run on commit 79bc73de
.
There were no benchmark performance regressions. 🎉
The full Conbench report has more details.