Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1946#issuecomment-55550076
In that case I'd propose merging this tentatively and if it causes issues
in the 1.2 dev/QA cycle we can revert it.
I dug around a bunch, it looks like there really isn't clear behavior here,
even within Linux variants because the code used to do this had to change a lot
for IPv6 support.There isn't an ordering specification anywhere I can find in
the internal API's in BSD/POSIX sockets that relate to this.
So it could be that this patch helps some and hurts others. As such, I'd be
inclined to try it out and rollback if we find anyone is hurt by this (if it's
not a strict win then we should bias towards the old behavior).
---
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]
