Github user tylerprete closed the pull request at:
https://github.com/apache/spark/pull/8992
---
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
Github user tylerprete commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-156224009
Yeah, sorry, life got busy and haven't made progress on this.
Realistically, I'm not sure I'll get to it right away. If I ever get around
to making it work,
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-152193338
Do you mind closing this PR?
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-151405945
I'm going to close this unless there's movement, given the discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-150181178
@tylerprete is this moving forward or should we close it at this point?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-147313013
Sounds good. IIRC it used more Jackson artifacts than Spark does, so
excluding all of them might leave it lacking a library it needs. If so the
solution would be to just
Github user tylerprete commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-147300513
@srowen I'll exclude jackson from json4s's transitive dependencies and see
if everything plays nice. That seems more tractable than tackling updating
jackson for all
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-146816853
@tylerprete I don't think this can proceed as-is because of the Jackson
change. Either you have to show it works with a down-graded Jackson, or
otherwise figure out how