Copilot commented on code in PR #12221:
URL: https://github.com/apache/gluten/pull/12221#discussion_r3357096109
##########
backends-velox/src-iceberg/test/scala/org/apache/gluten/execution/enhanced/VeloxIcebergSuite.scala:
##########
@@ -465,4 +465,75 @@ class VeloxIcebergSuite extends IcebergSuite {
)
}
}
+ ignore("disabled test") {
+ test("iceberg native write respects target file size bytes") {
+ withTable("iceberg_small_target_tbl") {
Review Comment:
The new test is effectively disabled in an unintended way: wrapping a
`test(...)` definition inside `ignore("disabled test") { ... }` means the inner
`test` is never registered (because the ignored test body is never executed).
If you want to disable this test, define it as `ignore("iceberg native write
respects target file size bytes") { ... }` directly (no nested `test`). If you
want it enabled, remove the outer `ignore` entirely.
--
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]
