[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-20 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r672775023 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -405,10 +405,18 @@ class JacksonParser( s

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-20 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r672775023 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -405,10 +405,18 @@ class JacksonParser( s

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-19 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r672775023 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -405,10 +405,18 @@ class JacksonParser( s

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-14 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r669635107 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala ## @@ -2924,6 +2924,55 @@ abstract class JsonSuit

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-14 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r669634117 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala ## @@ -2924,6 +2924,55 @@ abstract class JsonSuit

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-14 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r669624897 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala ## @@ -67,6 +86,8 @@ class FailureSafeParser[IN](

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-14 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r669623572 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala ## @@ -29,11 +29,30 @@ class FailureSafeParser[IN]

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-14 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r669487760 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -418,6 +426,19 @@ class JacksonParser( }

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-12 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r668068075 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -418,6 +426,19 @@ class JacksonParser( }

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666916933 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala ## @@ -2924,6 +2924,53 @@ abstract class JsonSuit

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666915867 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -418,6 +426,19 @@ class JacksonParser( }

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666915341 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -418,6 +426,19 @@ class JacksonParser( }

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666914473 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -399,16 +399,24 @@ class JacksonParser( val

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666912457 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -399,16 +399,24 @@ class JacksonParser( val

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-09 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666905445 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala ## @@ -2924,6 +2924,53 @@ abstract class JsonSuit

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-08 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666394738 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala ## @@ -2924,6 +2924,50 @@ abstract class JsonSuit

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-08 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666392613 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -404,6 +407,19 @@ class JacksonParser( }

[GitHub] [spark] cloud-fan commented on a change in pull request #33212: [SPARK-35912][SQL] Fix nullability of `spark.read.json`

2021-07-08 Thread GitBox
cloud-fan commented on a change in pull request #33212: URL: https://github.com/apache/spark/pull/33212#discussion_r666390792 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala ## @@ -404,6 +407,19 @@ class JacksonParser( }