Address comments

Author: HyukjinKwon

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala

@@ -126,12 +126,10 @@ class JacksonParser(
         case VALUE_STRING =>
           // Special case handling for NaN and Infinity.
           parser.getText match {
-            case "NaN" | "+NaN" | "-NaN" => Float.NaN
-            case "+INF" | "INF" | "+Infinity" | "Infinity" => Float.PositiveInfinity
-            case "-INF" | "-Infinity" => Float.NegativeInfinity
-            case other => Try(other.toFloat).getOrElse {
-              throw new RuntimeException(s"Cannot parse $other as FloatType.")
-            }
+            case "NaN" => Float.NaN
+            case "Infinity" => Float.PositiveInfinity
+            case "-Infinity" => Float.NegativeInfinity
+            case other => throw new RuntimeException(s"Cannot parse $other as FloatType.")
           }
       }
 
@@ -143,12 +141,10 @@ class JacksonParser(
         case VALUE_STRING =>
           // Special case handling for NaN and Infinity.
           parser.getText match {
-            case "NaN" | "+NaN" | "-NaN" => Double.NaN
-            case "+INF" | "INF" | "+Infinity" | "Infinity" => Double.PositiveInfinity
-            case "-INF" | "-Infinity" => Double.NegativeInfinity
-            case other => Try(other.toDouble).getOrElse {
-              throw new RuntimeException(s"Cannot parse $other as DoubleType.")
-            }
+            case "NaN" => Double.NaN
+            case "Infinity" => Double.PositiveInfinity
+            case "-Infinity" => Double.NegativeInfinity
+            case other => throw new RuntimeException(s"Cannot parse $other as DoubleType.")
           }
       }
 

sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala

@@ -1992,25 +1992,13 @@ class JsonSuite extends QueryTest with SharedSQLContext with TestJsonData {
 
   test("SPARK-18772: Parse special floats correctly") {
     val jsons = Seq(
-      """{"a": "+INF"}""",
-      """{"a": "INF"}""",
-      """{"a": "-INF"}""",
       """{"a": "NaN"}""",
-      """{"a": "+NaN"}""",
-      """{"a": "-NaN"}""",
       """{"a": "Infinity"}""",
-      """{"a": "+Infinity"}""",
       """{"a": "-Infinity"}""")
 
     // positive cases
     val checks: Seq[Double => Boolean] = Seq(
-      _.isPosInfinity,
-      _.isPosInfinity,
-      _.isNegInfinity,
-      _.isNaN,
       _.isNaN,
-      _.isNaN,
-      _.isPosInfinity,
       _.isPosInfinity,
       _.isNegInfinity)