Github user michalsenkyr commented on a diff in the pull request:
https://github.com/apache/spark/pull/20505#discussion_r165903346
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLImplicits.scala
---
@@ -165,11 +165,15 @@ abstract class SQLImplicits extends
LowPrioritySQLImplicits {
def newProductSeqEncoder[A <: Product : TypeTag]: Encoder[Seq[A]] =
ExpressionEncoder()
/** @since 2.2.0 */
- implicit def newSequenceEncoder[T <: Seq[_] : TypeTag]: Encoder[T] =
ExpressionEncoder()
+ implicit def newSequenceEncoder[T[_], E : Encoder]
--- End diff --
Looks like we are. I can add new methods and make the old ones not
implicit. That should fix MiMa. Although that might add to the clutter that's
already in this class. Is that OK?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]