Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18875#discussion_r137225157
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/jsonExpressions.scala
---
@@ -640,6 +644,7 @@ case class StructsToJson(
lazy val rowSchema = child.dataType match {
case st: StructType => st
case ArrayType(st: StructType, _) => st
+ case MapType(_: DataType, _: DataType, _) => child.dataType
--- End diff --
We can add a todo: `//TODO: support ArrayType(MapType)`, if we don't plan
to add the support in this PR.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
