Skip to content

Commit 99436d6

Browse files
author
wanghaihua
committed
Fix and add more test case.
1 parent cbb424f commit 99436d6

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

sql/core/src/test/scala/org/apache/spark/sql/execution/SparkSqlParserSuite.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -305,5 +305,6 @@ class SparkSqlParserSuite extends PlanTest {
305305
test("reset") {
306306
assertEqual("reset", ResetCommand(None))
307307
assertEqual("reset spark.test.property", ResetCommand(Some("spark.test.property")))
308+
assertEqual("reset #$a!", ResetCommand(Some("#$a!")))
308309
}
309310
}

sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ class SQLConfSuite extends QueryTest with SharedSQLContext {
115115
}
116116

117117
Seq("reset", s"reset ${SQLConf.GROUP_BY_ORDINAL.key}").foreach { resetCmd =>
118-
test("reset - public conf") {
118+
test(s"reset - public conf $resetCmd") {
119119
spark.sessionState.conf.clear()
120120
val original = spark.conf.get(SQLConf.GROUP_BY_ORDINAL)
121121
try {
@@ -133,7 +133,7 @@ class SQLConfSuite extends QueryTest with SharedSQLContext {
133133
}
134134

135135
Seq("reset", s"reset ${SQLConf.OPTIMIZER_MAX_ITERATIONS.key}").foreach { resetCmd =>
136-
test("reset - internal conf") {
136+
test(s"reset - internal conf $resetCmd") {
137137
spark.sessionState.conf.clear()
138138
val original = spark.conf.get(SQLConf.OPTIMIZER_MAX_ITERATIONS)
139139
try {
@@ -151,7 +151,7 @@ class SQLConfSuite extends QueryTest with SharedSQLContext {
151151
}
152152

153153
Seq("reset", s"reset $testKey").foreach { resetCmd =>
154-
test("reset - user-defined conf") {
154+
test(s"reset - user-defined conf $resetCmd") {
155155
spark.sessionState.conf.clear()
156156
try {
157157
assert(spark.conf.getOption(testKey).isEmpty)

0 commit comments

Comments
 (0)