This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 2f0340701f5c [SPARK-50243][SQL][TESTS][FOLLOWUP] Remove
`ARTIFACTS_SESSION_ISOLATION_ALWAYS_APPLY_CLASSLOADER` setting from
`AdaptiveQueryExecSuite`
2f0340701f5c is described below
commit 2f0340701f5c8c958324b5a801b69b3acc39a21f
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Mon Jan 6 15:39:13 2025 -0800
[SPARK-50243][SQL][TESTS][FOLLOWUP] Remove
`ARTIFACTS_SESSION_ISOLATION_ALWAYS_APPLY_CLASSLOADER` setting from
`AdaptiveQueryExecSuite`
### What changes were proposed in this pull request?
This PR is a follow-up to revert one line from the irrelevant test case of
`AdaptiveQueryExecSuite`.
- #49007
### Why are the changes needed?
`AdaptiveQueryExecSuite` has been a flaky test and was designed to repeat
the test logic many times. It's irrelevant to #49007 .
### Does this PR introduce _any_ user-facing change?
No, this is a revert to the original code.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #49374 from dongjoon-hyun/SPARK-50243.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala | 1 -
1 file changed, 1 deletion(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
index d298b98aaa8d..ad28fd5176d9 100644
---
a/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
@@ -749,7 +749,6 @@ class AdaptiveQueryExecSuite
// so retry several times here to avoid unit test failure.
eventually(timeout(15.seconds), interval(500.milliseconds)) {
withSQLConf(
- SQLConf.ARTIFACTS_SESSION_ISOLATION_ALWAYS_APPLY_CLASSLOADER.key ->
"true",
SQLConf.ADAPTIVE_EXECUTION_ENABLED.key -> "true",
SQLConf.NON_EMPTY_PARTITION_RATIO_FOR_BROADCAST_JOIN.key -> "0.5") {
// `testData` is small enough to be broadcast but has empty partition
ratio over the config.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
