Github user CodingCat commented on a diff in the pull request:
https://github.com/apache/spark/pull/14901#discussion_r77156607
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/SparkSQLExample.scala ---
@@ -203,7 +203,7 @@ object SparkSQLExample {
// No pre-defined encoders for Dataset[Map[K,V]], define explicitly
implicit val mapEncoder =
org.apache.spark.sql.Encoders.kryo[Map[String, Any]]
// Primitive types and case classes can be also defined as
- implicit val stringIntMapEncoder: Encoder[Map[String, Int]] =
ExpressionEncoder()
+ // implicit val stringIntMapEncoder: Encoder[Map[String, Any]] =
ExpressionEncoder()
--- End diff --
I am saying that the comments (L205) seem to indicate that, besides L204,
the encoder can be defined as L206 as well. Actually, L206 has the incorrect
type (you can find that by removing L204 and compile with the original L206)
so, I fixed the type and comment the line for successful compilation
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
