[GitHub] spark pull request: [SPARK-14989][BUILD] Upgrade Jackson from 2.5....

2016-04-29 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/12766#issuecomment-215876639 @JoshRosen yes, thank you for following up on this! On compatibility: there were some issues with 2.7.0 - 2.7.2, regarding type resolution, most of which

[GitHub] spark pull request: [SPARK-14222][BUILD] Remove jackson-module-sca...

2016-04-06 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/12213#issuecomment-206672520 @JoshRosen probably makes sense, but just in case it needs to be revisited Jackson's core databind has much more support for "referential types" (

[GitHub] spark pull request: [SPARK-14222][BUILD] Remove jackson-module-sca...

2016-04-06 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/12213#issuecomment-206566391 @JoshRosen ah. Yes, good point on usage via json4s. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-14222][BUILD] Remove jackson-module-sca...

2016-04-06 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/12213#issuecomment-206555763 I don't know enough about Spark internals to know if this makes sense, but there is a smaller alternative databinding, Jackson jr: https://githu

[GitHub] spark pull request: [SPARK-12269][Streaming][Kinesis] Update aws-j...

2016-01-28 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/10256#issuecomment-176310354 @srowen Ah. Yes, just wanted to mention it; I don't have enough context so there may be specific reasons for picking up particular versions. Glad to see ve

[GitHub] spark pull request: [SPARK-12269][Streaming][Kinesis] Update aws-j...

2016-01-28 Thread cowtowncoder
Github user cowtowncoder commented on the pull request: https://github.com/apache/spark/pull/10256#issuecomment-176300804 I know this issue is closed, but one suggestion I have would be to specify the latest Jackson 2.5 version, 2.5.5 instead of 2.5.3. While I understand caution in