incubator-gluten icon indicating copy to clipboard operation
incubator-gluten copied to clipboard

[OPPRO-223] Change to use partial avg in Velox

Open rui-mo opened this issue 3 years ago • 1 comments

What changes were proposed in this pull request?

Previously, we use sum and count to replace partial avg. But incompatible intermediate type was found for avg(int). Instead of adding a projection for casting bigint to double, we changed to use partial avg in Velox and extract the subfields from struct.

Depends on https://github.com/oap-project/velox/pull/38.

How was this patch tested?

under test

rui-mo avatar Aug 16 '22 06:08 rui-mo

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/gluten/issues

Then could you also rename commit message and pull request title in the following format?

[Gluten-${ISSUES_ID}] ${detailed message}

See also:

github-actions[bot] avatar Aug 16 '22 06:08 github-actions[bot]