Github user sadhen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22685#discussion_r224053894
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala ---
@@ -567,10 +567,10 @@ class KeyValueGroupedDataset[K, V] private[sql](
override def toString: String = {
val builder = new StringBuilder
val kFields = kExprEnc.schema.map {
- case f => s"${f.name}: ${f.dataType.simpleString(2)}"
+ f => s"${f.name}: ${f.dataType.simpleString(2)}"
--- End diff --
Should not be a partial function!
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
