[ 
https://issues.apache.org/jira/browse/SPARK-14222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15215286#comment-15215286
 ] 

Marcelo Vanzin commented on SPARK-14222:
----------------------------------------

bq. the extra verbosity required by other approaches is a feature, IMO, rather 
than a bug because it makes it much harder to accidentally break wire 
compatibility

I strongly disagree with that. Manual mapping is very, very, very error-prone, 
leads to things like our current event logs which don't map nicely to *any* 
language because fields have spaces in the names, and don't really integrate 
well with any tools (e.g. we check our public REST API using mima, which is not 
possible with our event logs because the mapping is all manual).

> Remove jackson-module-scala dependency or cross-publish Jackson for Scala 2.12
> ------------------------------------------------------------------------------
>
>                 Key: SPARK-14222
>                 URL: https://issues.apache.org/jira/browse/SPARK-14222
>             Project: Spark
>          Issue Type: Sub-task
>          Components: Build
>            Reporter: Josh Rosen
>
> In order to build Spark against Scala 2.12, we need to either remove our 
> jackson-module-scala dependency or cross-publish Jackson for Scala 2.12. 
> Personally, I'd prefer to remove it because I don't think we make extensive 
> use of it and because I'm not a huge fan of the implicit mapping between case 
> classes and JSON wire formats (the extra verbosity required by other 
> approaches is a feature, IMO, rather than a bug because it makes it much 
> harder to accidentally break wire compatibility).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to