[VL] Spark width_bucket function support

[gaoyangxiaozhu]

velox already support width_bucket function

#4039

How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

[github-actions]

Thanks for opening a pull request!

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

https://github.com/apache/incubator-gluten/issues

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

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

• Other pull requests

[github-actions]

Run Gluten Clickhouse CI

[gaoyangxiaozhu]

fail due to Velox width_bucket treat bucket_number <=0 as a user error (invalid parameter) https://github.com/facebookincubator/velox/blob/668d578420f20568e2f6c152c8eed77d3ad64c59/velox/functions/prestosql/Arithmetic.h#L384C1-L388C75 however spark would return null for this case https://github.com/apache/spark/blob/08c6bb9bf32f31b5b9870d56cc4c16ab97616da6/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala#L1749

@zhouyuan @PHILO-HE and @zhli1142015 , may basic c++ question, do you know how to modify velox widthBatch function to let it support return null value for those specifal cases as the result type already be set to int64_t ?

[FelixYBW]

hose specifal cases as the result type already be set to

Velox uses Presto implementation. You may need to create a new function for Velox/Spark, and pass the reference of null set to the function.

[github-actions]

Run Gluten Clickhouse CI

[github-actions]

Run Gluten Clickhouse CI

[gaoyangxiaozhu]

@rui-mo help review , thanks!

[rui-mo]

Thanks.