This is an automated email from the ASF dual-hosted git repository.
gurwls223 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 248fd4c67101 [SPARK-48342][FOLLOWUP][SQL] Remove unnecessary import in
AstBuilder
248fd4c67101 is described below
commit 248fd4c671018c5ea0be368b8b5a398313cd1713
Author: David Milicevic <[email protected]>
AuthorDate: Thu Jun 20 09:30:45 2024 +0900
[SPARK-48342][FOLLOWUP][SQL] Remove unnecessary import in AstBuilder
### What changes were proposed in this pull request?
In [SQL Scripting parser PR](https://github.com/apache/spark/pull/46665) we
didn't notice that import for `scala.collection.immutable.Seq` was added to
`AstBuilder` (probably accidentally by IntelliJ). While this import is not
unused per se, it is logically not needed (since Scala 2.13, `scala.Seq` is an
alias for `scala.collection.immutable.Seq`).
Anyways, no enforcement was there before and it makes sense to leave it as
such. It's not in any way required for SQL Scripting parser change.
### Why are the changes needed?
To remove accidentally added package enforcement that's not needed and
wasn't used before.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Already existing tests cover this change.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #47025 from davidm-db/sql_scripting_remove_import.
Authored-by: David Milicevic <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
.../src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala | 1 -
1 file changed, 1 deletion(-)
diff --git
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
index f716d632df8a..0d8bfe5f988d 100644
---
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
+++
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
@@ -20,7 +20,6 @@ package org.apache.spark.sql.catalyst.parser
import java.util.Locale
import java.util.concurrent.TimeUnit
-import scala.collection.immutable.Seq
import scala.collection.mutable.{ArrayBuffer, ListBuffer, Set}
import scala.jdk.CollectionConverters._
import scala.util.{Left, Right}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
