Skip to content

Commit b64383c

Browse files
sarutakdongjoon-hyun
authored andcommitted
[SPARK-48139][CONNECT][TESTS] Re-enable SparkSessionE2ESuite.interrupt tag
### What changes were proposed in this pull request? This PR proposes to re-enable `SparkSessionE2ESuite.interrupt tag`. This test was disabled in SPARK-48138 due to its flakiness. Based on my analysis, the flakiness was caused by the following issues. * SPARK-50889 * SPARK-53673 * SPARK-53339 Now all the issues above have been addressed for `4.2.0` so `SparkSessionE2ESuite.interrupt tag` is ready to be enabled. ### Why are the changes needed? For better test coverage. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? GA. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #54863 from sarutak/reenable-SparkSessionE2ESuite. Authored-by: Kousuke Saruta <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent 2bc6f75 commit b64383c

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

sql/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/SparkSessionE2ESuite.scala

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,7 @@ class SparkSessionE2ESuite extends ConnectFunSuite with RemoteSparkSession {
138138
assert(interrupted.length == 2, s"Interrupted operations: $interrupted.")
139139
}
140140

141-
// TODO(SPARK-48139): Re-enable `SparkSessionE2ESuite.interrupt tag`
142-
ignore("interrupt tag") {
141+
test("interrupt tag") {
143142
val session = spark
144143
import session.implicits._
145144

0 commit comments

Comments
 (0)