Github user CodingCat commented on the pull request:
https://github.com/apache/spark/pull/8533#issuecomment-139528532
@wulei-bj-cn , that's all in my side, thanks for your work
As @kayousterhout and @srowen pointed out, we have changed time and time
again on fetching the hostname/ip address of the server, which makes the
correctness of this very fragile...as a result, this patch may break the
assumption of other modules who need hostname/ip address (TaskLocation.scala
might be fine, not sure.....)
Let's wait for the comments from others who are more confident about the
changes we made around this.....
---
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]
