[GitHub] spark pull request #20884: [SPARK-23773][SQL] JacksonGenerator does not incl...

2018-06-29 Thread makagonov
Github user makagonov closed the pull request at: https://github.com/apache/spark/pull/20884 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #20884: [SPARK-23773][SQL] JacksonGenerator does not incl...

2018-03-23 Thread makagonov
Github user makagonov commented on a diff in the pull request: https://github.com/apache/spark/pull/20884#discussion_r176842732 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/json/JacksonGeneratorSuite.scala --- @@ -56,7 +56,7 @@ class JacksonGeneratorSuite

[GitHub] spark pull request #20884: [SPARK-23773][SQL] JacksonGenerator does not incl...

2018-03-22 Thread makagonov
GitHub user makagonov opened a pull request: https://github.com/apache/spark/pull/20884 [SPARK-23773][SQL] JacksonGenerator does not include keys that have null value for StructTypes ## What changes were proposed in this pull request? As stated in Jira, when `toJSON` is called