[VL] Aggregate window gets the wrong result in 1.2.0

[ccat3z]

Backend

VL (Velox)

Bug description

Aggregate window gets the wrong result

-- Data
(for { x <- 1 to 4500 } yield { (x,1) }).toDF("id", "order_num")
     
-- Query
SELECT
  id,
  SUM(num) OVER(ORDER BY num DESC) AS sum
FROM test

Expect: sum: 4500

Actual: sum: 4096 ... 4500

Spark version

None

Spark configurations

No response

System information

No response

Relevant logs

No response

[ccat3z]

cc @kecookier

[ccat3z]

This issue can be solved by disabling RowsStreamingWindowBuild. I guess that RowsStreamingWindowBuild might not be compatible with some aggregate case. I tried copying the logic of 'supportRowsStreaming()' from the update branch, but it didn't work. cc @JkSelf

[JkSelf]

@ccat3z I will look at this issue later. Thanks.

[JkSelf]

@ccat3z Can you help to verify facebookincubator/velox#10979 in your env? Thanks.

[ccat3z]

@ccat3z Can you help to verify facebookincubator/velox#10979 in your env? Thanks.

It works. Thanks.

[JkSelf]

Fixed in facebookincubator/velox#10979.