Rui Mo
Rui Mo
Verified on TPC-DS, but some fixes were needed for Velox backend. @taiyang-li Could you give me the access to push to this branch, so I can add them to this...
> Access added, you can push now. Thanks for that. But I found I still cannot push to this PR, so I opened one PR to your branch. Maybe you...
> @rui-mo Can you see why velox uts are failed? It is likely related to decimal types. Will check it.
hi @taiyang-li, all failed Velox unit tests were caused by a ProjectTransformer verification in [GlutenTestsTrait](https://github.com/oap-project/gluten/blob/main/gluten-ut/src/test/scala/org/apache/spark/sql/GlutenTestsTrait.scala#L170-L176) with below error. These tests were runnable but fallbacked into vanilla Spark for execution due...
@taiyang-li The PR from @PHILO-HE was merged, and rebase is needed for this PR. Thanks.
Another UT failure was found on Velox backend, will take a look. ``` - null *** FAILED *** org.apache.spark.SparkException: Job aborted due to stage failure: Task 1 in stage 8.0...
@taiyang-li The CI for Velox backend passed. Could you help change the Velox branch in build_velox.sh back as below? Then we can merge this PR. ``` VELOX_REPO=https://github.com/oap-project/velox.git VELOX_BRANCH=main ```
To fix the incorrectness of TPC-DS q54.
Verified locally on TPC-H and TPC-DS with correct result.
> please rebase to main. > > @rui-mo do you verified TPC-DS and TPCH on Velox Backend? Will do that today, thanks.