This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-connect-swift.git
The following commit(s) were added to refs/heads/main by this push:
new 7bb1e10 [SPARK-54998] Disable `ConstraintTests`
7bb1e10 is described below
commit 7bb1e10a914ac9a7a1a68987c96196ddef9618a1
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Mon Jan 12 10:05:26 2026 +0900
[SPARK-54998] Disable `ConstraintTests`
### What changes were proposed in this pull request?
This PR aims to disable `ConstraintTests` until Apache Iceberg supports
Apache Spark 4.1.0.
### Why are the changes needed?
We have had `Constraint` syntax test coverage for Spark 4.1.0+.
https://github.com/apache/spark-connect-swift/blob/2e2ef6e8f7ec162eba3b59abb91b39e65633d347/Tests/SparkConnectTests/ConstraintTests.swift#L32
However, Spark 4.1.1 starts to throw exceptions on `Constraint` syntax for
DSv1 tables after SPARK-54761.
- https://github.com/apache/spark/pull/53532
We need to revisit this later. SPARK-54999 is filed to re-enable this.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #278 from dongjoon-hyun/SPARK-54998.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
Tests/SparkConnectTests/ConstraintTests.swift | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/Tests/SparkConnectTests/ConstraintTests.swift
b/Tests/SparkConnectTests/ConstraintTests.swift
index 05568d9..177b3d7 100644
--- a/Tests/SparkConnectTests/ConstraintTests.swift
+++ b/Tests/SparkConnectTests/ConstraintTests.swift
@@ -22,8 +22,9 @@ import SparkConnect
import Testing
/// A test suite for new syntaxes from SPARK-51207 (SPIP: Constraints in DSv2)
-/// For now, only syntax test is here because Apache Spark 4.1 and the
corresponding Apache Iceberg is not released yet.
-@Suite(.serialized)
+/// For now, this suite is disabled due to SPARK-54761 because the
corresponding Apache Iceberg
+/// is not released yet. We will revise this to DSv2 table when Iceberg starts
to support Spark 4.1
+@Suite(.disabled("TODO(SPARK-54999): Re-enable ConstraintTests when Iceberg
supports Spark 4.1)"))
struct ConstraintTests {
@Test
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
