Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/7929#issuecomment-127992986
@srowen Suffered some network issue and couldn't get my comment posted
while merging this.
My intention was to merge the surgical first to see whether this fixes
various master builds, since it's kinda urgent. And TBH, I'm super confident
with the solution that only looks for 2.0.x. But yes, please feel free to open
a PR to add this, as it's a more general fix. With the surgical one merged
first, we can easily revert following experiments if things are not working as
expected.
(Another thing is that `Hadoop20SShims` is against Hadoop 0.20 with
security patch, while `Hadoop23Shims` is against 0.23. They are not for 2.0.x
and 2.3.x. The names are really confusing though.)
---
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]
