[GLUTEN-8229][VL] Don't rewrite collect_list/collect_set in window

[zml1206]

What changes were proposed in this pull request?

(Fixes: #8229)

How was this patch tested?

[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]

#8229

[zhztheplayer]

Hi, is the patch ready for review? I assume it may cause window fallback.

[zml1206]

Hi, is the patch ready for review? I assume it may cause window fallback.

It's already for review , I tested locally with VeloxWindowExpressionSuite.test("collect_list / collect_set").

[zhztheplayer]

Hi, is the patch ready for review? I assume it may cause window fallback.

It's already for review , I tested locally with VeloxWindowExpressionSuite.test("collect_list / collect_set").

My mistake. Somehow I didn't see the change in VeloxSparkPlanExecApi.scala. Thanks for the explanation.

[zhztheplayer]

Related to #5655