LuciferYang commented on code in PR #12981:
URL: https://github.com/apache/gluten/pull/12981#discussion_r3965913975
##########
backends-clickhouse/src-kafka/test/scala/org/apache/gluten/execution/kafka/ClickhouseGlutenKafkaScanSuite.scala:
##########
@@ -104,7 +104,7 @@ class ClickhouseGlutenKafkaScanSuite
}
// TODO: after rebase-25.12, failed with spark35 (0 did not equal 100000),
fix later
- testWithSpecifiedSparkVersion("GLUTEN-9681: test kafka data consistency",
"3.3") {
+ ignore("GLUTEN-9681: test kafka data consistency") {
Review Comment:
Done in 9b4261e, and applied to all four cases that were unpinned from 3.3
so they read the same way:
```scala
// Ignored: fails on Spark 3.5 after the ClickHouse 25.12 rebase (0 did not
equal 100000).
// TODO: fix and re-enable.
```
The reason was already in the file as a bare TODO, but nothing said it was
why the case is skipped. That was implicit in the old
`testWithSpecifiedSparkVersion(..., "3.3")` and got lost when the pin went away.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
