Copilot commented on code in PR #1819:
URL: https://github.com/apache/auron/pull/1819#discussion_r2684556702
##########
auron-spark-tests/spark33/src/test/scala/org/apache/auron/utils/AuronSparkTestSettings.scala:
##########
@@ -28,6 +28,19 @@ class AuronSparkTestSettings extends SparkTestSettings {
// See https://github.com/apache/auron/issues/1724
.exclude("string / binary substring function")
+ enableSuite[AuronDataFrameAggregateSuite]
+ // See https://github.com/apache/auron/issues/1840
+ .excludeByPrefix("collect functions")
+ // Added a new test to handle this issue
Review Comment:
The comment "Added a new test to handle this issue" is ambiguous. It's
unclear whether "this issue" refers to issue #1840 mentioned above or to the
SPARK-19471 test being excluded below. Consider clarifying the comment to
explicitly state that a custom version of the SPARK-19471 test has been added
to AuronDataFrameAggregateSuite with modified plan checks for Auron's native
aggregates.
```suggestion
// A custom version of the SPARK-19471 test has been added to
AuronDataFrameAggregateSuite
// with modified plan checks for Auron's native aggregates, so we
exclude the original here.
```
--
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]
