Skip to content

Commit 4d15f64

Browse files
ostronautcloud-fan
authored andcommitted
[SPARK-51219][SQL][TESTS][FOLLOWUP] ShowTablesExec remove ArrayImplicits`
### What changes were proposed in this pull request? Related to #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. ### Why are the changes needed? To fix failing scala 2.12 compilation isssu. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Existing init tests and actions run. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #50008 from ostronaut/features/ShowTablesExec-remove-ArrayImplicits. Authored-by: Dima <[email protected]> Signed-off-by: Wenchen Fan <[email protected]>
1 parent 030e691 commit 4d15f64

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/ShowTablesExec.scala

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ import org.apache.spark.sql.catalyst.util.StringUtils
2525
import org.apache.spark.sql.connector.catalog.{CatalogV2Util, Identifier, TableCatalog}
2626
import org.apache.spark.sql.connector.catalog.CatalogV2Implicits.NamespaceHelper
2727
import org.apache.spark.sql.execution.LeafExecNode
28-
import org.apache.spark.util.ArrayImplicits._
2928

3029
/**
3130
* Physical plan node for showing tables.
@@ -50,8 +49,7 @@ case class ShowTablesExec(
5049

5150
private def isTempView(ident: Identifier, catalog: TableCatalog): Boolean = {
5251
if (CatalogV2Util.isSessionCatalog(catalog)) {
53-
session.sessionState.catalog
54-
.isTempView((ident.namespace() :+ ident.name()).toImmutableArraySeq)
52+
session.sessionState.catalog.isTempView(ident.namespace() :+ ident.name())
5553
} else false
5654
}
5755
}

0 commit comments

Comments
 (0)