Skip to content

Commit b24aedf

Browse files
committed
isDefined instead of ne(None).
1 parent da88a85 commit b24aedf

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,11 +260,11 @@ class OracleIntegrationSuite extends DockerJDBCIntegrationSuite with SharedSQLCo
260260
assert(parentPlan.isInstanceOf[WholeStageCodegenExec])
261261
val node = parentPlan.asInstanceOf[WholeStageCodegenExec]
262262
val metadata = node.child.asInstanceOf[RowDataSourceScanExec].metadata
263-
// The "PushedFilters" part should be exist in Dataframe's
263+
// The "PushedFilters" part should exist in Dataframe's
264264
// physical plan and the existence of right literals in
265265
// "PushedFilters" is used to prove that the predicates
266266
// pushing down have been effective.
267-
assert(metadata.get("PushedFilters").ne(None))
267+
assert(metadata.get("PushedFilters").isDefined)
268268
assert(metadata("PushedFilters").contains(dt.toString))
269269
assert(metadata("PushedFilters").contains(ts.toString))
270270

sql/core/src/test/scala/org/apache/spark/sql/test/SQLTestUtils.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ import org.apache.spark.sql.catalyst.plans.PlanTest
3939
import org.apache.spark.sql.catalyst.plans.logical.LogicalPlan
4040
import org.apache.spark.sql.catalyst.util._
4141
import org.apache.spark.sql.execution.FilterExec
42+
import org.apache.spark.sql.internal.SQLConf
4243
import org.apache.spark.util.{UninterruptibleThread, Utils}
4344

4445
/**

0 commit comments

Comments
 (0)