Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3321#issuecomment-63397633
This looks good to me. I tried doing a cherry-pick myself and it looks
like the original merge conflicts were caused by the ConnectionManager being
moved to the `nio` subpackage in 1.2 and due to
73bf3f2e0c03216aa29c25fea2d97205b5977903 / #2593 and a couple of other commits
that weren't present in branch-1.2, which caused the `tryFailure` /
`trySuccess` code to be missing from the diff context.
---
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]
