This is an automated email from the ASF dual-hosted git repository.
wenchen pushed a commit to branch branch-4.0
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.0 by this push:
new 60e1d4adefd7 [SPARK-51219][SQL][TESTS][FOLLOWUP] ShowTablesExec`
remove `ArrayImplicits`
60e1d4adefd7 is described below
commit 60e1d4adefd782fbfeba705bc956ee584eaa53b8
Author: Dima <diman...@gmail.com>
AuthorDate: Thu Feb 20 11:03:37 2025 +0800
[SPARK-51219][SQL][TESTS][FOLLOWUP] ShowTablesExec` remove `ArrayImplicits`
Related to https://github.com/apache/spark/pull/49144. scala 2.12 is
failing with `ArrayImplicits`, which is in use for `ShowTablesExec.isTempView`
method. This PR removes `org.apache.spark.util.ArrayImplicits._` from
`ShowTablesExec` and uses default Seq instead.
To fix failing scala 2.12 compilation isssu.
No
Existing init tests and actions run.
No.
Closes #50008 from ostronaut/features/ShowTablesExec-remove-ArrayImplicits.
Authored-by: Dima <diman...@gmail.com>
Signed-off-by: Wenchen Fan <wenc...@databricks.com>
(cherry picked from commit 4d15f6465b7a163d04dcec7ec748c78e1217d0fb)
Signed-off-by: Wenchen Fan <wenc...@databricks.com>
---
.../apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
a/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala
b/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala
index f314dfd79119..8ccd126b4b22 100644
---
a/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala
+++
b/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala
@@ -25,7 +25,6 @@ import org.apache.spark.sql.catalyst.util.StringUtils
import org.apache.spark.sql.connector.catalog.{CatalogV2Util, Identifier,
TableCatalog}
import
org.apache.spark.sql.connector.catalog.CatalogV2Implicits.NamespaceHelper
import org.apache.spark.sql.execution.LeafExecNode
-import org.apache.spark.util.ArrayImplicits._
/**
* Physical plan node for showing tables.
@@ -50,8 +49,7 @@ case class ShowTablesExec(
private def isTempView(ident: Identifier, catalog: TableCatalog): Boolean = {
if (CatalogV2Util.isSessionCatalog(catalog)) {
- session.sessionState.catalog
- .isTempView((ident.namespace() :+ ident.name()).toImmutableArraySeq)
+ session.sessionState.catalog.isTempView((ident.namespace() :+
ident.name()).toSeq)
} else false
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
